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

fix: Modernize expression_lib #111

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

MarcoGorelli
Copy link
Contributor

@MarcoGorelli MarcoGorelli commented Oct 2, 2024

closes #110

  • polars.utils is gone, but LIB = Path(__file__).parent is all that's needed now
  • polars.type_aliases is also gone, so it's necessary to redefine IntoExprColumn
  • parse_into_expr is no longer needed, register_plugin_function now takes care of that

@MarcoGorelli MarcoGorelli force-pushed the modernise-expression-lib branch 6 times, most recently from 83ab6f8 to dd789d8 Compare October 2, 2024 09:04
@MarcoGorelli MarcoGorelli marked this pull request as ready for review October 2, 2024 09:07
@MarcoGorelli MarcoGorelli force-pushed the modernise-expression-lib branch 2 times, most recently from 9f00247 to 473fdbc Compare October 2, 2024 09:09
@ritchie46 ritchie46 merged commit d426148 into pola-rs:main Oct 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No module named 'polars.utils'
2 participants