You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
### Issue Description
I am not really sure what the expectations are for handling non-ISO strings as an indexer for a DTI. In the examples above it seems acceptable for the /stop/ argument in the slice to have the day first but the /start/ argument does not assume this
### Expected Behavior
Not sure - maybe we should just disallow non-ISO string formats?
### Installed Versions
'3.0.0.dev0+681.g434fda08cf'
The text was updated successfully, but these errors were encountered:
Hey @WillAyd, agree that this is confusing behavior. I took a look at the documentation and there is no clarification on how time strings should be handled in this scenario. This leads to a lot of ambiguity especially in an altered example of the one you provided above:
ser = pd.Series(range(365), index=pd.date_range(start="2024-01-01", freq="D", periods=365))
ser.loc["7/1/2024":"8/1/2024"] # could be interpreted as M/D/YYYY or D/M/YYYY
My opinion would be the same as yours, enforce iso format for date strings to limit user confusion and update documentation for clarity, although this might have backward compatibility issues as we are tightening the interface. If we are looking for a lighter compromise, we could add a warning for when non-iso string formats are used. Any thoughts?
thanks for the ping - agree on moving towards this being iso-like only
it's one thing to allow flexibility when parsing csvs (say), where users don't necessarily have control over what format their data's in - but it's another when using loc, when users should be in control of what they pass in
Pandas version checks
I have checked that this issue has not already been reported.
I have confirmed this bug exists on the latest version of pandas.
I have confirmed this bug exists on the main branch of pandas.
Reproducible Example
The text was updated successfully, but these errors were encountered: