-
Notifications
You must be signed in to change notification settings - Fork 87
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
Intermediate release 0.7.3.1 #1263
Conversation
nicl-nno
commented
Mar 5, 2024
- Minor fixes
- Modified int tests
👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app Simply create a new comment in this PR that says: @docu-mentor run and I will start my analysis. I only look at what you changed @docu-mentor run doc/ README.md In this example, I'll have a look at all files contained in the "doc/" |
All PEP8 errors has been fixed, thanks ❤️ Comment last updated at |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1263 +/- ##
==========================================
- Coverage 79.85% 79.85% -0.01%
==========================================
Files 146 146
Lines 10028 10030 +2
==========================================
+ Hits 8008 8009 +1
- Misses 2020 2021 +1 ☔ View full report in Codecov by Sentry. |
Только хотел залить свои фиксы интеграционников) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
По поводу удаления последнего assert - мне тоже кажется он лишним. Учитывая кол-во обученных пайплайнов, логично, что дошли до сходимости и там может не быть разницы из-за сидов
/fix-pep8 |