-
Notifications
You must be signed in to change notification settings - Fork 20
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
Changes for Tau-embedded samples #18
base: HTT_skim_v1
Are you sure you want to change the base?
Conversation
@DennRoy, there is a possibility to redefine parameters in the samples' yaml files. I think having a separate |
Thanks! I was able to successfully redo the test by adding those parameters to the yaml files, also testing the new pfns prefix argument. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @DennRoy ! Modifications look fine. Please update the RunKit sub-module to the current RunKit/main.
@DennRoy do you plan any other updates? If not, I am ok with merging this PR |
The people from KIT are still looking into my suggested customization. They say it looks alright, but didn't get to directly compare the outputs yet with what they have. |
It all works fine so far, but there's still one more issue related to some trigger bit information (IDs which were 15 for Taus before are not always that value anymore in nAODv12). I've been told that this could be related to some special trigger, which is also not yet implemented. The people at KIT are still working on this. |
Some additional details: I'm actually unable to process embedded samples for 2017. There's an error about the unavailability of DeepTau v2.1 branches in the input (only v2 seems to be available in there). The other eras (2016, 2016_HIMP, 2018) work fine though. But apparently there's no problem for the KIT people when producing v9 NanoAODs. |
@DennRoy thanks for the update.
|
In order to run Tau-embedded samples, an additional customization is required. This customization is based on these instructions, but contain some modifications in order for this to work with a newer CMSSW version and NanoAOD v12.
The easiest way to process embedded samples is to run them separately from data or MC, by using a separate overseer config.The embedded samples can be run along with all other data or MC samples. All necessary modifications needed for the embedded samples are implemented in the yaml file for the embedded samples. The differences compared to the nominal config are the path to the additional customization, and points to a different DBS instance ("phys03" instead of "global").This PR is not ready to be merged, as I would like embedding experts to sign off on the customizations added here.