Specific variable plotting bugfix#99
Conversation
- Rename MT2_blbl → MT2_blbl1 (l1b1+l2b2 pairing) for clarity - Add MT2_blbl1/2_delta_phi and MT2_blbl1/2_ptratio observables - Update histogram configs, global variables list, and DNN feature configs accordingly - Remove jj_dphi from variables list (was never defined, would cause crash)
|
@cms-flaf-bot please test |
|
pipeline#15187777 started |
|
pipeline#15187777 passed |
kandrosov
left a comment
kandrosov
left a comment
There was a problem hiding this comment.
remove variables from ci_custom. In this way all variables from global yaml will be tested by CI to avoid similar bugs in future.
Got it! |
Wait, do you mean user_custom.yaml, or ci_custom.yaml? User_custom is where I have all the MT2 vars, I've not touched ci_custom and seems to just store DNN variables. |
|
remove these lines from ci_custom https://github.com/cms-flaf/HH_bbWW/blob/main/config/ci_custom.yaml#L19-L30 |
This is a local custom config file, correct? Does the CI test from the bot take this into account? If this isn't true, then I'm not sure I follow how the CI "knows" that all variables should be tested. |
|
@cms-flaf-bot please test |
|
pipeline#15227418 started |
|
pipeline#15227418 passed |
|
@cms-flaf-bot please test |
|
pipeline#15228233 started |
|
pipeline#15228233 failed |
|
@cms-flaf-bot please test |
|
pipeline#15229581 started |
|
@cms-flaf-bot please test |
|
pipeline#15269859 started |
|
pipeline#15269859 passed |
|
@kandrosov do you think this new "run on all variables" is acceptable? It moved the CI test from ~30 minutes (https://gitlab.cern.ch/cms-flaf/flaf_integration/-/pipelines/15227418) to almost 2 hours (https://gitlab.cern.ch/cms-flaf/flaf_integration/-/pipelines/15269859). |
4 participants
added histograms for more MT2 variables