egenn / rtemis

Advanced Machine Learning and Visualization
https://rtemis.org
GNU General Public License v3.0
137 stars 19 forks source link

polars: can it be moved to your "rtemis-extra" package? #57

Closed HenrikBengtsson closed 9 months ago

HenrikBengtsson commented 9 months ago

The polars package:

Because of this, the dependency on polars:

My suggestion is to remove polars as a dependency on rtemis. If it can be incorporated via that other package where you moved other dependencies (rtemis-extra?), that would be much better.

If polars is removed, then it should be straightforward to set up package checks via GitHub Actions.

egenn commented 9 months ago

Agreed. My plan with packages like this has been to wait and watch their status until I start preparing for rtemis 1.0 release and CRAN submission, at which point I'd move all problematic packages to rtemis-extra. Might as well do that now, so GitHub Actions checks can get started. For now, I think I'll just comment out the polars part.