aimclub / FEDOT

Automated modeling and machine learning framework FEDOT
https://fedot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
619 stars 84 forks source link

enh: `giotto_tda` and `risper` -> other_requirements #1259

Closed aPovidlo closed 4 months ago

aPovidlo commented 4 months ago

Summary

Modules giotto_tda and risper requires to migrate to other_requirements for advantage Time Series, due to fact, that risper has dependence of C++.

nicl-nno commented 4 months ago

@valer1435 это реально или потребуем каких-то нетривиальных действий? И когда ты на винду у себя ставил - это воспроизводилось?

migrate to other_requirements Наверное лучше какой-то отдельный файл в духе tda_ts_requirements.txt

valer1435 commented 4 months ago

Воспроизводимость зависит от того, стоит ли build tools. Вообще такие зависимости действительно неприятны. Наверное стоит делать проверку на наличие у юзера этого пакета перед выбором пресета.

nicl-nno commented 4 months ago

Добавишь тогда таковую?

valer1435 commented 4 months ago

https://github.com/aimclub/FEDOT/pull/1261