-
Notifications
You must be signed in to change notification settings - Fork 44
Add jet and fatJet jsons for 2022-preEE and 2022-postEE #210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Hi @izisopou, thanks a lot. It would be consistent if we can keep the same directory structure as that of json-pog e.g.
What is happening for FlavourQCD for the low pT regions:
This we see for both years and jet algos |
|
Hi @ravindkv , If we retain the same directory structure as in jsonPOG-integration, we would lose the ability to keep all JSON versions for a specific campaign in a single, consolidated location. Furthermore, directly copying that structure would effectively duplicate the entire jsonPOG-integration repository here. If full duplication is indeed the goal, then we can do that, but I do not think it is needed; we can just continue using the jsonPOG-integration repo. Rather than simply moving the Regarding the JR versions, we can think this a bit more. For example, if we have For FlavorQCD I am not sure. We will have to check the corresponding txt file as this has not changed in this specific PR. I applied the JSON files to a 2025C dataset. Could this be the cause of this problem? |
|
Hi @izisopou , thanks for the thoughts.
same value (16e-03) for pt <30. It is just that the plot is zoomed too much that is why we see big error bars. |
|
Hi @ravindkv , Thank you for your reply and for checking the FlavorQCD text file, everything seems to be reasonable indeed. If you agree, I can open a MR in jsonPOG-Integration with the updated jsons. Regarding this repo, I agree with your comment about the inconsistency. Technically, we would need a JERCDatabase for the JSON files, and maybe that was the reason why the jsonFiles folder was not used so far in the first place. On one hand, keeping an archive of all JSON files will definitely be useful, on the other hand I worry we may confuse analyzers with this repository. If we proceed with merging this PR we may have to explicitly make clear that this folder is only an archive and the JSON files to be used should be taken from jsonPOG-Integration (hence my update on the README file). Another option would be to transfer the jsonFiles folder in a private JERC gitlab repo just for us and JME, and keep jsonPOG-Integration as the only public place for JERC JSON files. |
|
Hi @izisopou, thanks a lot.
|

In this PR we upload the jet_jerc.json.gz and fatJet_jerc.json.gz files for
Summer22_22Sep2023andSummer22EE_22Sep2023.Summer22_22Sep2023_V2JEC andSummer22_22Sep2023_JRV1JER.Summer22_22Sep2023_V3JEC andSummer22_22Sep2023_JRV1JERSummer22EE_22Sep2023_V2JEC andSummer22EE_22Sep2023_JRV1JER.Summer22EE_22Sep2023_V3JEC andSummer22EE_22Sep2023_JRV1JERMoving from the JCV2 to the JCV3 versions, we fixed an asymptotic discontinuity in the
L2RelativeJEC files. Related PRs are: #206 and #207 .In the files attached, we compare the JCV2_JRV1 (currently in jsonPOG-Integration) vs JCV3_JRV1 (to be uploaded to jsonPOG-Integration) JSONs:
out_JetJSON_2022EE_BeforeVsAfterAsymptFix_PtEta.pdf
out_JetJSON_2022_BeforeVsAfterAsymptFix_PtEta.pdf
out_FatJetJSON_2022EE_BeforeVsAfterAsymptFix_PtEta.pdf
out_FatJetJSON_2022_BeforeVsAfterAsymptFix_PtEta.pdf
As expected, everything is identical except for the specific eta bins in the
L2Relativewhere the fits were reproduced in order to fix the asymptotic discontinuities.The JCV3_JRV1 versions are expected to replace the JCV2_JRV1 versions in jsonPOG-Integration, once this PR is approved and merged.
cc: @ravindkv