functime-org / functime

Time-series machine learning at scale. Built with Polars for embarrassingly parallel feature extraction and forecasts on panel data.
https://docs.functime.ai
Apache License 2.0
975 stars 52 forks source link

feat!: Move all feature extractors into namespace only + speedup Lempel Ziv #113

Closed abstractqqq closed 8 months ago

abstractqqq commented 8 months ago

Improved performance for Lempel Ziv by applying the change to Hashbrown suggested by Ritchie

BREAKING CHANGE:

  1. The functime.feature_extraction.tsfresh module is merged into functime.feature_extractor. This has 2 advantages: (a). Simpler, keeping things in one place and shorter import path. (b) This avoids a potential circular import. If we write plugin code, the code needs to be registered in a namespace (in feature_extractor.py), but for Series to benefit from it, we have to import the code to tsfresh.py. But feature_extractor module imports from tsfresh.py... Hence the circular import. Merging them together solves this issue
vercel[bot] commented 8 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
functime-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 7, 2023 9:07am
abstractqqq commented 8 months ago

@topher-lo Any idea why Vercel is failing?

topher-lo commented 8 months ago
Screenshot 2023-11-06 023920