Skip to content
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

Migrate to polars #37

Closed
sco1 opened this issue Nov 14, 2024 · 1 comment · Fixed by #39
Closed

Migrate to polars #37

sco1 opened this issue Nov 14, 2024 · 1 comment · Fixed by #39
Labels
enhancement New feature or request

Comments

@sco1
Copy link
Owner

sco1 commented Nov 14, 2024

Not an urgent task, but would be helpful to align with my processing libraries for other devices.

@sco1 sco1 added the enhancement New feature or request label Nov 14, 2024
@sco1
Copy link
Owner Author

sco1 commented Nov 15, 2024

Initial migration is finished to the extend of currently written testing. A few more small things left:

  • Rewrite log trimming
  • Add tests for log trimming
  • Ensure column ordering is consistent
  • See if there's a way to bring back time-interval based rolling windowing. We were able to handle this in pandas because the index was a timedelta but there isn't an analogous approach available in polars.

@sco1 sco1 mentioned this issue Nov 15, 2024
@sco1 sco1 closed this as completed in #39 Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant