Closed kmohrman closed 2 years ago
Merging #283 (1105001) into master (b0c0916) will decrease coverage by
2.86%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #283 +/- ##
==========================================
- Coverage 29.06% 26.19% -2.87%
==========================================
Files 42 42
Lines 6699 6699
==========================================
- Hits 1947 1755 -192
- Misses 4752 4944 +192
Flag | Coverage Δ | |
---|---|---|
unittests | 26.19% <ø> (-2.87%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
topcoffea/modules/corrections.py | 0.00% <0.00%> (-35.39%) |
:arrow_down: |
topcoffea/modules/objects.py | 0.00% <0.00%> (-20.88%) |
:arrow_down: |
topcoffea/modules/selection.py | 0.00% <0.00%> (-6.96%) |
:arrow_down: |
topcoffea/modules/dataDrivenEstimation.py | 68.64% <0.00%> (-3.39%) |
:arrow_down: |
topcoffea/modules/GetValuesFromJsons.py | 32.14% <0.00%> (-1.79%) |
:arrow_down: |
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 b0c0916...1105001. Read the comment docs.
As suggested by @btovar, this PR updates the WQ timeout (for the wq CI test) from 300s to 400s.