Skip to content

Date transformations #180

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 29, 2025
Merged

Date transformations #180

merged 2 commits into from
Oct 29, 2025

Conversation

@paco-barreras
Copy link
Collaborator

Fixes #127

@paco-barreras
filters: add to_yyyymmdd; fix to_timestamp TZ handling for strings; m…
daae22e 
…ake is_within honor traj_cols; tests refocused on main APIs; add edge-case tests (empty/bad inputs, nullable types); validate freq in _q_series
paco-barreras
paco-barreras commented Oct 29, 2025
View reviewed changes
Copy link
Collaborator Author

@paco-barreras paco-barreras left a comment

Choose a reason for hiding this comment

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

This passes the tests in test_filters.py which were previously broken

@paco-barreras paco-barreras merged commit 46aa445 into main Oct 29, 2025
@paco-barreras paco-barreras deleted the date_transformations branch October 29, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@carolineychen8 carolineychen8 Awaiting requested review from carolineychen8

@andresmondragont andresmondragont Awaiting requested review from andresmondragont

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Create function in filters.py that converts dates (and datetimes?) to numerical values (int YYYYMMDD)

2 participants

@paco-barreras