Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC: Update infer_dtype docstring #61111

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

gm-oo9
Copy link
Contributor

@gm-oo9 gm-oo9 commented Mar 12, 2025

@gm-oo9 gm-oo9 marked this pull request as ready for review March 12, 2025 09:57
@gm-oo9 gm-oo9 mentioned this pull request Mar 12, 2025
1 task
@@ -5108,8 +5108,8 @@ def _warn_about_deprecated_aliases(name: str, is_period: bool) -> str:
warnings.warn(
f"\'{name}\' is deprecated and will be removed "
f"in a future version, please use "
f"\'{c_PERIOD_AND_OFFSET_DEPR_FREQSTR.get(name)}\'"
f" instead.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you undo the changes in this file? They seem unrelated

@@ -1573,6 +1573,7 @@ def infer_dtype(value: object, skipna: bool = True) -> str:

Notes
-----
- The value parameter must be an iterable; scalar inputs are not supported.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- The value parameter must be an iterable; scalar inputs are not supported.

This is already described in the docstring

@mroeschke mroeschke added the Docs label Mar 12, 2025
@gm-oo9 gm-oo9 force-pushed the fix-infer-dtype-docstring branch from 7b2b6fd to 16a0d6c Compare March 13, 2025 14:17
@gm-oo9 gm-oo9 requested a review from mroeschke March 13, 2025 15:36
@mroeschke mroeschke added this to the 3.0 milestone Mar 13, 2025
@mroeschke mroeschke merged commit ddd0aa8 into pandas-dev:main Mar 13, 2025
42 checks passed
@mroeschke
Copy link
Member

Thanks @gm-oo9

@gm-oo9 gm-oo9 deleted the fix-infer-dtype-docstring branch March 13, 2025 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: pd.api.types.infer_dtype on scalar input
2 participants