FEDOT icon indicating copy to clipboard operation
FEDOT copied to clipboard

Published 20 hours ago • verified publisher icon aimclub

Suggested minor improvements for FEDOT API

Open MorrisNein opened this issue 2 years ago • 4 comments

Some convenience improvements should be performed at any time:

  • [ ] Allow Fedot.fit to get path to features and target via Path objects.
  • [ ] Write explicit docstring for Fedot.fit.
  • [ ] Use generic type FeaturesType in Fedot.fit annotation.
  • [ ] Rename stop_after_n_generations -> early_stopping_generations here. Also see: https://github.com/nccr-itmo/FEDOT/issues/899
  • [ ] https://github.com/nccr-itmo/FEDOT/issues/873
  • [ ] https://github.com/nccr-itmo/FEDOT/issues/874 (Possibly linked to the above)

MorrisNein avatar Sep 23 '22 13:09 MorrisNein

Еще стоит переименовать stop_after_n_generations в early_stopping_generations в докстринге

valer1435 avatar Sep 26 '22 15:09 valer1435

Предложу еще заменить имена пресетов в constants.py на нормальный Enum

gkirgizov avatar Oct 07 '22 09:10 gkirgizov

Предложу еще заменить имена пресетов в constants.py на нормальный Enum

На мой взгляд не стоит enum-ы выносить в какие-то внешние интерфейсы. На практике в python-фреймворках так делают редко. Ну и тянет за собой лишние импорты.

nicl-nno avatar Oct 07 '22 19:10 nicl-nno

Предложу еще заменить имена пресетов в constants.py на нормальный Enum

На мой взгляд не стоит enum-ы выносить в какие-то внешние интерфейсы.

К слову, необязательно обязывать внешний интерфейс использовать Enum. А вообще у Python, кмк, есть идеальный выход для данной ситуации:

image

Т.е. пока мы сравниваем через ==, для нас строковая переменная будет равнозначна элементу перечисления.

MorrisNein avatar Oct 08 '22 08:10 MorrisNein