-
Notifications
You must be signed in to change notification settings - Fork 113
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
Remove flaky tests #342
Remove flaky tests #342
Conversation
Given that we've hit #340 quite a few times now, do we want to just skip the multi-objective NSGA2 tests here until that's sorted? |
Codecov Report
@@ Coverage Diff @@
## main #342 +/- ##
===========================================
- Coverage 83.97% 72.29% -11.68%
===========================================
Files 22 22
Lines 3363 3364 +1
===========================================
- Hits 2824 2432 -392
- Misses 539 932 +393 see 9 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Agreed
The RTD failure is tracked at readthedocs/readthedocs#10290, we should be able to safely ignore it. |
I will have @eirikurj merging this because I do not have the authority to override the required docs test |
I'm gonna exercise my power before they're gone :) |
Purpose
This test has caused problems in the past and now it's caused one conda-forge build to fail. I think it's probably best to just remove this time, don't think it's adding much value anyways.
Expected time until merged
Not urgent.
Type of change
Testing
Checklist
flake8
andblack
to make sure the Python code adheres to PEP-8 and is consistently formattedfprettify
or C/C++ code withclang-format
as applicable