-
-
Notifications
You must be signed in to change notification settings - Fork 18.3k
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
DEPR: deprecate errors='ignore' in to_datetime and make output dtype predictable #54467
Comments
+1 |
Likewise for |
@MarcoGorelli I am willing to take up this issue but I was hoping if I can get more info into this, if we deprecate errors='ignore' then shall it return a warning that it is deprecated? |
yup, that's right! |
okay, then I will take up this issue. |
take |
@MarcoGorelli @Cyddharth-Gupta For my applications,
and the expected output: 0 object 0 float64 |
that's a different function, this issue is just about to_Datetime |
Thanks @MarcoGorelli for replying. Since @mroeschke mentioned that it is the same for |
what is the recommended refactor for a line of code like this that gets a warning in pytest?
|
i'd suggest to wrap it in |
The below might work
|
Feature Type
Adding new functionality to pandas
Changing existing functionality in pandas
Removing existing functionality in pandas
Problem Description
currently, if you do
to_datetime(inputs)
, you don't really know what the dtype of the output will be. It could beIndex
orDatetimeIndex
Deprecating parsing mixed offsets goes part of the way to addressing this
Feature Description
Can we go all the way there, and deprecate
errors='ignore'
? Then, if the computation succeeds, then you getDatetimeIndex
(potentially with someNaT
s iferrors='coerce'
)Alternative Solutions
none that I can think of
Additional Context
there's talk about query optimisation in pandas, and greater predictability should help with that
The text was updated successfully, but these errors were encountered: