-
Notifications
You must be signed in to change notification settings - Fork 122
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
feat: add datetime
selector
#1822
Merged
Merged
Changes from 2 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
9d9b735
feat: datetime selector
FBruzzesi aedb28c
unit test
FBruzzesi 438eb27
use .dt.convert_time_zone first
FBruzzesi ef1d271
maybe with backport
FBruzzesi 4f05167
forgot pyproject ;)
FBruzzesi e5493e1
fail pyarrow on windows, use replace_time_zone
FBruzzesi 7899103
fail for old pandas
FBruzzesi b7c21f7
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi 6b0b006
add is_order_dependent arg
FBruzzesi 26ed283
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi 57672ec
it passes :)
FBruzzesi e5ad0c0
force pytest to run with utc
FBruzzesi 319624f
Update pyproject.toml
FBruzzesi 0edd7ca
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi cb4823f
fix up
FBruzzesi 36e215f
merge main
FBruzzesi 9c8fbfb
arrow type hint
FBruzzesi 11124f7
rethink the logic
FBruzzesi 4010a6f
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi a57dc1d
add pyspark & duckdb, test timezone.utc
FBruzzesi 51c3d81
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi b789c00
add 's' in default time_units, trim docstrings
FBruzzesi 40f3d8f
rm kwargs={dtypes}
FBruzzesi a9d9f37
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi b2d8818
whops
FBruzzesi 610d553
Merge branch 'main' into feat/selectors-by-datetime
FBruzzesi 242d2fa
Merge branch 'main' into feat/selectors-by-datetime
MarcoGorelli 22eee5d
feedback adjustments
FBruzzesi c166dff
more docstring trimming
FBruzzesi 64f7f5c
merge main and rm comment
FBruzzesi b998459
merge main
FBruzzesi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For dask, the selector works, but the cast fails with: