ramp-workflow icon indicating copy to clipboard operation
ramp-workflow copied to clipboard

[WIP] Generative regression (workflow, scores, cross validation, prediction type)

Open kegl opened this issue 5 years ago • 15 comments

kegl avatar Dec 02 '19 10:12 kegl

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

lgtm-com[bot] avatar Dec 02 '19 11:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 02 '19 12:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 03 '19 14:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 11 '19 13:12 lgtm-com[bot]

Codecov Report

Merging #193 (ea0853c) into master (212720f) will decrease coverage by 0.90%. The diff coverage is 79.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 Impacted file tree graph

@@            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.

codecov[bot] avatar Dec 12 '19 08:12 codecov[bot]

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

lgtm-com[bot] avatar Dec 18 '19 17:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 26 '19 19:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Jan 13 '20 18:01 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 04 '20 21:12 lgtm-com[bot]

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

lgtm-com[bot] avatar Dec 04 '20 21:12 lgtm-com[bot]

This pull request introduces 1 alert when merging cccd6300152170c7952c65ad35e7b2966b5c79b3 into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com

new alerts:

  • 1 for Unguarded next in generator

lgtm-com[bot] avatar Dec 04 '20 22:12 lgtm-com[bot]

This pull request introduces 1 alert when merging e08620289859c6f87246f1e5c7ab85a8f1bfe3ba into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com

new alerts:

  • 1 for Unguarded next in generator

lgtm-com[bot] avatar Dec 06 '20 13:12 lgtm-com[bot]

This pull request introduces 1 alert when merging 442504363de6041abf58fcd08620cca6c79565cb into ec8835ef68de2389b54e7e730f1967a17b6100db - view on LGTM.com

new alerts:

  • 1 for Unguarded next in generator

lgtm-com[bot] avatar Mar 05 '21 15:03 lgtm-com[bot]

This pull request introduces 1 alert when merging c00e178beb57fd37232585e81e55c39650876f1a into 74686f73391191e42d25cb0eb7d11d2f53571a1c - view on LGTM.com

new alerts:

  • 1 for Unguarded next in generator

lgtm-com[bot] avatar Mar 05 '21 16:03 lgtm-com[bot]

This pull request introduces 1 alert when merging ea0853c60747901fa073abc988044e7776d6a2eb into 74686f73391191e42d25cb0eb7d11d2f53571a1c - view on LGTM.com

new alerts:

  • 1 for Unguarded next in generator

lgtm-com[bot] avatar Mar 05 '21 17:03 lgtm-com[bot]