-
Notifications
You must be signed in to change notification settings - Fork 85
765 Disable stdout output & tqdm in failing tests #766
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## master #766 +/- ##
==========================================
- Coverage 87.70% 87.61% -0.10%
==========================================
Files 186 185 -1
Lines 12875 12885 +10
==========================================
- Hits 11292 11289 -3
- Misses 1583 1596 +13
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
02a93d4
to
831ad0e
Compare
Hello @gkirgizov! 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! 🍻 Comment last updated at 2022-08-05 09:05:18 UTC |
a9a6a42
to
0676f4f
Compare
28ed93b
to
4523d76
Compare
1149601
to
7f12e11
Compare
8a5361f
to
8ddb143
Compare
The main issue was because of this line: https://github.com/nccr-itmo/FEDOT/blob/7eb49b15c0f614493c5c94f5ac0ca5c6405d15b7/fedot/core/operations/operation.py#L80 |
8ddb143
to
2f72772
Compare
c0f1bb6
to
78adf2b
Compare
Не по твоим изменениям, но ты вроде начал наводить порядок и перенаправлять выходные сообщения в логер, предлагаю в рамках этого PR рассмотреть включение этих мест в логирование: |
Конечно, если для этого не понадобится дополнительный рефакторинг |
Fixes #765