Closed kasyanovse closed 5 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:
Code in this pull request still contains PEP8 errors, please write the /fix-pep8
command in the comments below to create commit with automatic fixes.
Attention: 30 lines
in your changes are missing coverage. Please review.
Comparison is base (
5e726e9
) 80.05% compared to head (8f895c3
) 79.84%.
Files | Patch % | Lines |
---|---|---|
...erations/topological/fast_topological_extractor.py | 30.23% | 30 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/fix-pep8
Хотелось бы тест на то, что фичи получаются +- те же, что и в обычном
Хотелось бы тест на то, что фичи получаются +- те же, что и в обычном
Здесь из топологии генерируются другие фичи, поэтому смысла в этом нет. Сравнение предсказаний для lagged
-topo
-ridge
на картинке. Я бы не сказал, что есть принципиальные отличия, однако можно сказать что fast_topo
не уловило низкочастотные составляющие. Это жертва ради скорости, но если нужно, то качество можно улучшить, докинув к квантилям еще и стат фичи.
This is a 🙋 feature or enhancement.
Summary
Ускоренная версия топологических фич (в 30 раз). От обычных топологических фич отличаются достаточно сильно:
Context
Inspired by https://github.com/aimclub/FEDOT/pull/1241.