-
Notifications
You must be signed in to change notification settings - Fork 59
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: Rustify lempel_ziv_complexity #107
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@messense can you shed some light on how we can ensure this best? I thought maturin build always put the shared library in the same folder as the python code? |
@abstractqqq I expect you have your file names incorrect. It is important that the name in the Cargo.toml, the python package and the rust lib match. |
I think the file isn't generated in the right location on windows... I did the same thing and got it working on Linux. It is working now |
By default yes, but it can be changed via In this case it's named |
@abstractqqq Apart from the conflicts (nothing too serious), I think this is good to go. |
Implemented the first Polars Plugin: lempel_ziv_complexity. We now have both eager and lazy version for this feature too ( and is super fast )
How to test:
Definitely need to review this process and try to improve this process. @topher-lo