martin1tab commented 2 years ago

Add the possibility to run hyperopt on multiple data sets in the same ramp kit by specifying --data-label

codecov[bot] commented 2 years ago

Codecov Report

Merging #295 (554e818) into master (aee6274) will increase coverage by 0.02%. The diff coverage is 88.46%.

@@            Coverage Diff             @@
##           master     #295      +/-   ##
==========================================
+ Coverage   82.97%   82.99%   +0.02%     
==========================================
  Files         137      137              
  Lines        4680     4693      +13     
==========================================
+ Hits         3883     3895      +12     
- Misses        797      798       +1

Impacted Files	Coverage Δ
rampwf/hyperopt/cli/hyperopt.py	`0.00% <0.00%> (ø)`
...terfaces/header_in_files/classifier_kit/problem.py	`92.59% <87.50%> (ø)`
rampwf/hyperopt/hyperopt.py	`98.64% <100.00%> (+0.02%)`	:arrow_up:
rampwf/hyperopt/tests/test_hyperparameter.py	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update aee6274...554e818. Read the comment docs.

kegl commented 2 years ago

Awesome, thank you @martin1tab .

@agramfort if no objection from the Saclay side, we'll merge this next week. It only touches hyperopt so no core functions that you use.

paris-saclay-cds / ramp-workflow

Supporting data label for hyperopt #295

Codecov Report