Rabbit tensorwriter for direct theory fit of sigmaUL#677
Merged
davidwalter2 merged 4 commits intoWMass:mainfrom Mar 25, 2026
Merged
Rabbit tensorwriter for direct theory fit of sigmaUL#677davidwalter2 merged 4 commits intoWMass:mainfrom
davidwalter2 merged 4 commits intoWMass:mainfrom
Conversation
Collaborator
|
I still don't understand what the input of the script is: is it the rabbit tensor (e.g., ZMassDilepton.hdf5) or is it the corrections file from SCETlib+DYTurbo (wremnants-data/data/TheoryCorrections/*.pkl.lz4) |
Collaborator
Author
Hi @kdlong the inputs are:
the output is a tensor for rabbit to the generator-level extraction. |
Collaborator
davidwalter2
left a comment
There was a problem hiding this comment.
The feedRabbitTheory script looks very good, I have a few comments to make it more integrated with the rest of the framework.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relies on #671
The previous script, feedRabbitTheory.py, was creating tensors for rabbit for sigmaUL, as well as for W and the Ai's. The latter two -- W and Ai -- were duplicate efforts to what setupRabbit.py already does, since it's in fiducial phase space, they have to be read from MiNNLO. The sigmaUL, on the other hand, is in full lepton phase space, so it can be compared directly to the generators' predictions and variations.
The new structure is to have a single feedRabbitSigmaUL.py, and all it does is create the rabbit tensor for sigmaUL directly from the TheoryCorrection files.
The sigmaUL tensor from feedRabbitSigmaUL.py and the W,Ai tensors from setupRabbit.py can be combined, as different cards, if one wants to do a simultaneous fit.