Closed Bogdan-Wiederspan closed 3 months ago
@Bogdan-Wiederspan What would need to be done for this PR to leave the WIP phase?
Nothing needs to be added @riga. I simply forgot to switch the state to "ready to review"
Actually not completely true, I am still including updated values using the GenXSecAnalyzer directly instead of cmsdb and there will be a few new processes to include once the run2 campaigns are merged (e.g. several multibosons decay channels like ZZ->QQ)
Updated values are now fully included with the corresponding logs and some logs expected to be needed after the merge, only the additional values which will come with the processes added with the merge are missing.
resolved Merge conflicts with master :)
Question (resolved): If we have a $\pm 2$% uncert is it noted in the scinum version we use as uncert=2j
or uncert=0.02j
? A few xsecs use the first notation
I tested it in the latest version of scinum and the latter would be correct, but I wasn't sure if the notation was different in older scinum versions.
I made a commit to correct these and can push it if the second notation turns out to be correct :sweat_smile:
Nice, thanks for spotting these
Question (resolved): If we have a % uncert is it noted in the scinum version we use as
uncert=2j
oruncert=0.02j
? A few xsecs use the first notation
0.02j
is indeed the right way to do it 👍
@nprouvost @mafrahm Are the two comments above resolved?
This PR adds all xsecs for Run2