Open jayceslesar opened 1 year ago
Hi @jayceslesar! 👋🏼
Glad to hear you are again willing to contribute to this package! 🚀
We are currently in the progress of altering the data-interface to something more array-alike, see #154. This PR will ease the integration with tsdownsample
; a tool that will give us significant speedups regarding the data aggregation. As such, a lot of code changes lie ahead of us. I think adding mypy
is a great feature; however, I would postpone it until this integration is performed!
However, there is some work that can be done.
layoutUpdater
component => https://github.com/predict-idlab/trace-updater/pull/9. However, my JavaScript knowledge is still not optimal; and can certainly make use of an extra pair of eyes; so you could review / benchmark the PR. FYI: yesterday we made a new release 0.8.3.1
(to which you contributed) 🎉
Looking forward to hearing back from you, Jonas
haha my javascript is also not optimal but am happy to take a peek -- a rangeslider feature would be super cool. WIll play around with that notebook when I have some time as well
Not sure if this is wanted or not but I was playing around with this and would either need to add a lot of
type: ignores
to the existing code or drop support for python 3.7 in the version where mypy is included as a few of the stubs needed to correctly type out some code do not support 3.7Happy to hack on this if wanted