ramp-workflow
ramp-workflow copied to clipboard
[WIP] Generative regression (workflow, scores, cross validation, prediction type)
This pull request introduces 8 alerts when merging 75721513193e6d81848bb23229d0fd6bc9b243ae into f48b024642c0756d9e12717067cbca224e34b9ce - view on LGTM.com
new alerts:
- 2 for Unused local variable
- 2 for Unused import
- 1 for Unnecessary pass
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 8 alerts when merging 814b598f856bce7db17d67f480f4060976ef69d4 into f48b024642c0756d9e12717067cbca224e34b9ce - view on LGTM.com
new alerts:
- 2 for Unused local variable
- 2 for Unused import
- 1 for Unnecessary pass
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 7 alerts when merging 7c68e928793b9faae00475b2936bc5cd0c062675 into 0d837ef16208423e6a3ce83da8ac2e596c9be44e - view on LGTM.com
new alerts:
- 2 for Unused import
- 1 for Unnecessary pass
- 1 for Unused local variable
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 7 alerts when merging 5860913d4c72b08338fe8054a091074587b2dfdc into c1b4273bb95542cf469dd83575d554bf5f2480af - view on LGTM.com
new alerts:
- 2 for Unused import
- 1 for Unnecessary pass
- 1 for Unused local variable
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
Codecov Report
Merging #193 (ea0853c) into master (212720f) will decrease coverage by
0.90%
. The diff coverage is79.33%
.
:exclamation: Current head ea0853c differs from pull request most recent head 2b9e240. Consider uploading reports for the commit 2b9e240 to get more accurate results
@@ Coverage Diff @@
## master #193 +/- ##
==========================================
- Coverage 83.05% 82.15% -0.91%
==========================================
Files 137 154 +17
Lines 4704 5794 +1090
==========================================
+ Hits 3907 4760 +853
- Misses 797 1034 +237
Impacted Files | Coverage Δ | |
---|---|---|
rampwf/utils/cli/blend.py | 0.00% <0.00%> (-70.97%) |
:arrow_down: |
rampwf/workflows/el_nino.py | 29.16% <33.33%> (ø) |
|
rampwf/cvs/time_series.py | 33.07% <36.89%> (+16.40%) |
:arrow_up: |
rampwf/prediction_types/base.py | 81.81% <50.00%> (-5.28%) |
:arrow_down: |
rampwf/utils/cli/testing.py | 62.50% <50.00%> (ø) |
|
rampwf/utils/generative_regression.py | 61.73% <61.73%> (ø) |
|
rampwf/workflows/generative_regressor.py | 67.11% <67.11%> (ø) |
|
rampwf/score_types/r2.py | 72.72% <72.72%> (ø) |
|
rampwf/utils/cli/show.py | 76.40% <73.33%> (ø) |
|
rampwf/workflows/ts_feature_extractor.py | 81.39% <79.16%> (+64.72%) |
:arrow_up: |
... and 79 more |
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 212720f...2b9e240. Read the comment docs.
This pull request introduces 8 alerts when merging 5f68f906d7a8b762569317fe6aec39b41a41c407 into c1b4273bb95542cf469dd83575d554bf5f2480af - view on LGTM.com
new alerts:
- 2 for Unused local variable
- 2 for Unused import
- 1 for Unnecessary pass
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 7 alerts when merging d77df0a2c81586eacc6be6f79bbf67098e5d6e09 into c1b4273bb95542cf469dd83575d554bf5f2480af - view on LGTM.com
new alerts:
- 2 for Unused local variable
- 1 for Unnecessary pass
- 1 for Unused import
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 7 alerts when merging 9bc65784c9b89740a1fc1ab4f869cd41481bdd04 into c1b4273bb95542cf469dd83575d554bf5f2480af - view on LGTM.com
new alerts:
- 2 for Unused local variable
- 1 for Unnecessary pass
- 1 for Unused import
- 1 for Explicit export is not defined
- 1 for Nested loops with same variable
- 1 for Unguarded next in generator
This pull request introduces 5 alerts and fixes 9 when merging deba54d4f4b0fc9df9b410cc8f126c09f08df905 into 67a26b3e70c87fd2bf0cea3c893e87c11739c634 - view on LGTM.com
new alerts:
- 4 for Explicit export is not defined
- 1 for Unguarded next in generator
fixed alerts:
- 5 for Implicit string concatenation in a list
- 2 for Module is imported with 'import' and 'import from'
- 1 for Module is imported more than once
- 1 for Unreachable code
This pull request introduces 5 alerts when merging deba54d4f4b0fc9df9b410cc8f126c09f08df905 into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com
new alerts:
- 4 for Explicit export is not defined
- 1 for Unguarded next in generator
This pull request introduces 1 alert when merging cccd6300152170c7952c65ad35e7b2966b5c79b3 into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com
new alerts:
- 1 for Unguarded next in generator
This pull request introduces 1 alert when merging e08620289859c6f87246f1e5c7ab85a8f1bfe3ba into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com
new alerts:
- 1 for Unguarded next in generator
This pull request introduces 1 alert when merging 442504363de6041abf58fcd08620cca6c79565cb into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com
new alerts:
- 1 for Unguarded next in generator
This pull request introduces 1 alert when merging c00e178beb57fd37232585e81e55c39650876f1a into 74686f73391191e42d25cb0eb7d11d2f53571a1c - view on LGTM.com
new alerts:
- 1 for Unguarded next in generator
This pull request introduces 1 alert when merging ea0853c60747901fa073abc988044e7776d6a2eb into 74686f73391191e42d25cb0eb7d11d2f53571a1c - view on LGTM.com
new alerts:
- 1 for Unguarded next in generator