Updates to support systematics without running out of memory
There are a few changes here, mostly to make sure we can run the systematic variations on the grid.
-
Use a service to share instances of the GNN. This is particularly needed when we run systematic variations. -
Move the jet calibration stuff out of the dumpster Alg, thus preventing one calibration tool per systematic -
Use public calibration tools which are shared between all the systematics -
Add an option to run the calibration after the GNN (currently it runs before) to evaluate how much this matters
-
-
Add back more systematics -
Add an option to raise the frequency of updates in eventLoopHeartBeat.txt
Reworking where the jet calibration happens made me realize that the jet systematics are a bit more subtle than I'd originally thought: we normally apply the flavor tagging on uncalibrated jets, but meanwhile jetmet provides systematics on calibrated jets. So to vary the inputs to flavor tagging we first have to calibrate the jets, which isn't what we do in production. My plan is to add an "uncalibrated" systematic (run the calibration after the GNN) to see how big an effect this is.
Might be interesting to @vhomsak.
This MR requires releases with with atlas/athena!71323 (merged), which should be 25.2.12 and 25.0.6. I expect the CI to fail until 25.2.12 is pushed to the gitlab registry.
Merge request reports
Activity
added 1 commit
- 2ae1a522 - factor out jet systematics, try to think more carefully about how this actually will work
mentioned in issue #144
- Resolved by Nikita Ivvan Pond
That's nice. I have however overcome the solution by reverting to loop-submission 1grid-submission = 1 systematic variation, and have already produced rich_ip retagged samples for GN2v01 and GN2v01FlipSimple
mentioned in merge request !681 (closed)
@npond I think you are the backup reviewer for my stuff while we don't have a Sam. How does this look to you?
added 16 commits
-
093b7610...9bcf690a - 6 commits from branch
atlas-flavor-tagging-tools:main
- 68c8e5c2 - use service to share NNs
- 1d2422b0 - release bumps
- a563b63b - factor out jet systematics, try to think more carefully about how this actually will work
- 82623f4a - make calibration after tagging work
- b1fb6fdd - add production systematic
- e3ac9fca - add back more systematics
- 16e65946 - lower print interval
- cff9c6c7 - fixes for new Athena version
- 97b034a4 - fix for flake8
- 5b23392b - update config name
Toggle commit list-
093b7610...9bcf690a - 6 commits from branch