Closed bryates closed 2 years ago
Merging #297 (8986953) into master (ee80d0d) will decrease coverage by
11.56%
. The diff coverage is4.44%
.
@@ Coverage Diff @@
## master #297 +/- ##
===========================================
- Coverage 35.82% 24.26% -11.57%
===========================================
Files 46 47 +1
Lines 7034 7117 +83
===========================================
- Hits 2520 1727 -793
- Misses 4514 5390 +876
Flag | Coverage Δ | |
---|---|---|
unittests | 24.26% <4.44%> (-11.57%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
analysis/topEFT/topeft.py | 9.40% <ø> (-72.87%) |
:arrow_down: |
topcoffea/modules/corrections.py | 0.00% <0.00%> (-55.15%) |
:arrow_down: |
topcoffea/modules/parsejec.py | 0.00% <0.00%> (ø) |
|
topcoffea/modules/datacard_tools.py | 71.79% <100.00%> (ø) |
|
topcoffea/modules/selection.py | 0.00% <0.00%> (-83.43%) |
:arrow_down: |
topcoffea/modules/objects.py | 0.00% <0.00%> (-59.35%) |
:arrow_down: |
topcoffea/modules/HistEFT.py | 61.98% <0.00%> (-0.24%) |
: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 ee80d0d...8986953. Read the comment docs.
This PR introduces a reduced set of JEC. The script
topcoffea/modules/parsejec.py
is used to take all the JEC we've decided to add in quadrature (e.g.BBEC1
andBBEC1_{year}
) and produces new correction txt files. This will currently add 5 new nuisance parameters to our workspace:FlavorQCD
,BBEC1
,Absolute
,RelativeBal
, andRelativeSample
Note:FlavorQCD
is overwritten with the proper flavor component (e.g. b-jets will useFlavorPureBottom
)