aimclub / FEDOT

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

1110 integration tests fix #1112

Closed IIaKyJIuH closed 1 year ago

IIaKyJIuH commented 1 year ago

Closes #1110

Main:

Others:

aim-pep8-bot commented 1 year ago

Hello @IIaKyJIuH! 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:

Comment last updated at 2023-08-11 13:29:50 UTC
IIaKyJIuH commented 1 year ago

Настроил во всех тестах один seed. Скорректировал время проведения тестов. Часть тестов удалось детерминировать, но есть ещё над чем работать.

Возможно, нужно в некоторых местах выбирать n_jobs=1

codecov[bot] commented 1 year ago

Codecov Report

Merging #1112 (37767f1) into master (8f1dcd7) will increase coverage by 0.17%. Report is 1 commits behind head on master. The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master    #1112      +/-   ##
==========================================
+ Coverage   78.53%   78.71%   +0.17%     
==========================================
  Files         131      131              
  Lines        9359     9362       +3     
==========================================
+ Hits         7350     7369      +19     
+ Misses       2009     1993      -16     
Files Changed Coverage Δ
fedot/api/api_utils/metrics.py 91.66% <ø> (ø)
fedot/api/fedot_cli.py 0.00% <0.00%> (ø)
fedot/api/main.py 56.03% <ø> (ø)
..._implementations/models/ts_implementations/cgru.py 29.10% <0.00%> (+0.21%) :arrow_up:
fedot/core/pipelines/pipeline.py 95.23% <0.00%> (-0.58%) :arrow_down:
fedot/core/operations/evaluation/gpu/clustering.py 51.35% <50.00%> (ø)
fedot/core/operations/evaluation/gpu/common.py 46.93% <50.00%> (ø)
fedot/api/api_utils/data_definition.py 84.14% <100.00%> (+26.45%) :arrow_up:
fedot/core/composer/composer_builder.py 85.71% <100.00%> (-0.96%) :arrow_down:
fedot/core/data/data.py 71.28% <100.00%> (ø)
... and 7 more

... and 6 files with indirect coverage changes

IIaKyJIuH commented 1 year ago

Удалось заставить проходить все тесты. Правда, это получается не всегда, попробую ещё подумать. SEED и N_JOBS как будто не влияют) Зато отчётливо проявляются проблемы с разными версиями питона.

nicl-nno commented 1 year ago

N_JOBS как будто не влияют

В рамках запуска на гитхабе или вообще?

IIaKyJIuH commented 1 year ago

N_JOBS как будто не влияют

В рамках запуска на гитхабе или вообще?

Вообще. Я имел в виду, что n_jobs не влияет на однозначность выполнения тестов. Более того, он явно замедляет выполнение тестов, не знаю точно на сколько.