-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update pyproject.toml #67
Conversation
I think this is a safe change - lttbc has been updated to 0.2.1
Hi, Thanks for your contribution! 😄 I would love to merge it into main, but there seems to be a problem with lttbc 0.2.1 (see #42). The compiled files use a numpy version that is rather recent, resulting in warnings for the current numpy versions plotly-resampler supports. I'm interested to hear your thoughts about this! Cheers, PS: @jonasvdd is currently looking into implementing & compiling a more optimized version of lttbc 🚀 |
Another evidence of lttbc's problematic behavior is that a strange error occurs when you try to deploy the configuration to binder:
So I very much appreciate the attempt to replace this package with the potentially more efficient fast-lttb. |
Hi, Some problems with As @jvdd already mentioned, I'm currently looking into implementing the core components of EfficientLTTB in c, see this repo. However, I cannot estimate how long this will take (have some other deadlines / trips coming up the next weeks), as I do not have much experience in compiling platform-agnostic python bindings for c code (which seems one of the major problems now with @zxweed, from your error it seems that fyi: a colleague of mine @mcourteaux is also looking into a really optimized version of these algorithms, leveraging the SIMD vector instruction set, see this repo |
I specified numpy 1.21 in the requirements and it successfully installed above in the log. Versions 0.1.9 of lttbc were left from trying to find one that worked out of the three - no luck. |
@zxweed, If you look at the original |
Also, for what it's worth, here's what I did to make lttbc install in a conda environment on an m1 mac, since there are not binaries published for osx/arm:
|
Hi @bbradbury In this PR #84, I am to add my own LTTB implementation to plotly-resampler with the aim of (1) reducing install errors and (2) adding optimized implementations of this algorithm. Feel free to take a look / give some thoughts if you have time. |
I think this is a safe change - lttbc has been updated to 0.2.1