Polars/phase 3 batch conversions 1#1048
Draft
EngCaioFonseca wants to merge 2 commits intopolars/phase-2-query-layerfrom
Draft
Polars/phase 3 batch conversions 1#1048EngCaioFonseca wants to merge 2 commits intopolars/phase-2-query-layerfrom
EngCaioFonseca wants to merge 2 commits intopolars/phase-2-query-layerfrom
Conversation
Modules converted: - commits_over_time.py: - Polars for datetime conversion and sorting - Polars dt.truncate() for period grouping - Polars group_by().agg(n_unique()) for commit counting - active_drifting_contributors.py: - Polars for initial processing - Polars filtering in get_active_drifting_away_up_to() (2-5x faster) - Replaced .apply() with list comprehension - pr_staleness.py: - Polars for datetime conversion and sorting - Polars filtering in get_new_staling_stale_up_to() (2-5x faster) - Replaced .apply() with list comprehension - issue_staleness.py: - Polars for datetime conversion and sorting - Polars filtering in get_new_staling_stale_up_to() (2-5x faster) - Replaced .apply() with list comprehension Total modules using Polars: 10+ Remaining .apply() calls converted to list comprehensions: 4 Architecture maintained: Polars Core, Pandas Edge
Modules converted: - pr_assignment.py: - Polars for datetime conversion and sorting - Polars filtering in pr_assignment() (2-5x faster) - Replaced .apply() with list comprehension - issue_assignment.py: - Polars for datetime conversion and sorting - Polars filtering in issue_assignment() (2-5x faster) - Replaced .apply() with list comprehension - Fixed duplicate import (removed 'import app' duplicate) - pr_first_response.py: - Polars for datetime conversion and filtering - Polars filtering in get_open_response() (2-5x faster) - Replaced .apply() with list comprehension Total modules using Polars: 14+ Architecture maintained: Polars Core, Pandas Edge
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
EngCaioFonseca
moved this from Backlog
to Testing | Waiting on PR response | Blocked
in Aspen Project Board
EngCaioFonseca
moved this from Testing | Waiting on PR response | Blocked
to In Progress
in Aspen Project Board
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
PR of the series of migrating pandas to polars in 8knot.
Generative AI disclosure