Closed kasyanovse closed 6 months ago
Hello @kasyanovse! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
299ffba
) 79.47% compared to head (fe4d61a
) 79.93%. Report is 2 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
- Сейчас подбор окна происходит при каждом вызове transform. впринципе ок, думаю можно оставить подбор для каждого фолда, но не слишеом ли то долго?
Почему при каждом вызове? Я сейчас посмотрел, после выбора параметры сохраняются в узле.
- Как-то можно посмотреть настоящую длину окна, а не параметр 0?
Да, в lagged
хранится настоящая длина окна после обучения.
- Как проверял эффективность? Действительно ли стало лучше?
https://github.com/aimclub/FEDOT/pull/1227 в описании PR есть таблички с оценкой изменения положения пайплайна внутри поколения после мутации.
Кстати тюнер будет игнорировать длину окна или нет?
Кстати тюнер будет игнорировать длину окна или нет?
Не будет. Он как раз и дотюнивает окно после композиции.
Method from https://github.com/aimclub/FEDOT/pull/1186.
lagged
node.