optuna
optuna copied to clipboard
Deselect integration tests in Tests CI
Motivation
Currently, samplers_tests/test_samplers.py
and importance_tests/test_init.py
require integration libraries.
Description of the changes
This PR will add an integration
marker and test integration samplers / importance-evaluators in the integration tests CI.
Please merge #3983 first.
Codecov Report
Merging #4013 (ff9f5b6) into master (8917a51) will increase coverage by
0.01%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #4013 +/- ##
==========================================
+ Coverage 90.10% 90.12% +0.01%
==========================================
Files 162 162
Lines 12746 12746
==========================================
+ Hits 11485 11487 +2
+ Misses 1261 1259 -2
Impacted Files | Coverage Δ | |
---|---|---|
optuna/integration/botorch.py | 98.26% <0.00%> (+0.86%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This pull request has not seen any recent activity.
I updated this PR. We can test basic features using pytest tests -m "not integration"
and integrations using pytest tests -m "integration"
.
@c-bata Could you review this PR?
This pull request has not seen any recent activity.
@not522 Any update on this friendly reminder :)
I updated the PR. Please take another look.
@not522 Sorry again for the delay and thank you for your huge effort :bow: There seems to be some conflicts due to #4150. Could you resolve it?
LGTM! I've just enabled auto-merge so that it will be merged after passed all required tests :pray: Thank you for the update.