-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
years-only in date_range uses current date #6961
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
Comments
I think this is simply an ambiguity, you can (and should specify the start or end date more fully) e.g. maybe could somehow provide a better inference on the default date though if its a 'partial' date. |
sorry..realized the other issue is closed (sort of punted on it). as I said, feel free to try to address. |
Going to be covered in #7599. Result after #7599The problem is caused by a path which uses today's date as default. Being fixed to use first day of the year (and specifying
Other datetime parse function results (for ref)Single year is parsed as the first day of the year in most function already (not changed by #7599).
|
Great! I did some ad-hoc testing on your PR and confirm it works as expected. Thanks! |
ok, so in #7599 just fix up the release notes if you would |
As this seems to be fixed in 0.15.0, should we just close this issue? |
really? can u a) see if u can find a refence to an issue which closed this ? if it works can u put a test that validates? |
ahh ok |
I ran this on master current as of 2014-04-20 or so and was surprised as to the result:
I would have expected that to give me the month-ends in 2014, not a year's worth of month-ends starting from today.
Seems to be related to #3944?
The text was updated successfully, but these errors were encountered: