Closed kasyanovse closed 11 months ago
А есть оценка насколько это ускоряет?
Тесты на произвольных данных. исправленный код находится в функции prepare_target
.
Старый вариант.
Новый вариант.
Merging #1139 (9410f15) into master (8f1dcd7) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #1139 +/- ##
=======================================
Coverage 78.53% 78.53%
=======================================
Files 131 131
Lines 9359 9360 +1
=======================================
+ Hits 7350 7351 +1
Misses 2009 2009
Files Changed | Coverage Δ | |
---|---|---|
...lementations/data_operations/ts_transformations.py | 78.42% <100.00%> (+0.05%) |
:arrow_up: |
Здорово! Да наверное в тесте нет нужды, время сложно оценивать.
One line in code in lagged implementation spends more than 95% of lagged implementation evaluation time (in my case). Fixed it.