-
Notifications
You must be signed in to change notification settings - Fork 148
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
enable possibility to apply different set of cuts to Muon and Global Muon Tracks #1725
Conversation
REQUEST FOR PRODUCTION RELEASES:
This will add The following labels are available |
"matchChi2MaxMFT": "45", | ||
"diMuonTimeCut": "100", | ||
"fullHistos": "0", | ||
"GID" : "MFT-MCH,MFT-MCH-MID" |
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.
@lmassacr In order to have the possibility to evaluate the tracks matching fraction between MFT and MCH-MID, the GID parameter should be modified as
"GID" : "MFT-MCH,MCH-MID,MFT-MCH-MID"
This is to have MCH-MID tracks with the same kinematic cuts as the MFT-MCH-MID ones.
In case one needs to reduce the number of QC objects that are produced by the task, I would propose to eventually remove the MFT-MCH
tracks from the selection:
"GID" : "MCH-MID,MFT-MCH-MID"
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.
Hi @aferrero2707 thanks for the comment.
I just did the modification with "GID" : "MFT-MCH,MCH-MID,MFT-MCH-MID" to follow what you did also for the synch QC.
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.
@lmassacr actually I just realized that MFT-MCH
in the GID list is useless without adding MCH
as well, because you need the set of MCH-only plots with the same cuts as MFT-MCH
for the comparisons.
If the number of plots becomes too big, you would need to restrict to the tracks with MID:
"GID" : "MCH-MID,MFT-MCH-MID"
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.
Hi @aferrero2707,
For the moment I commited the config file adding the MCH standalone tracks with the same cuts as MCH-MFT tracks.
I will see if there are concerns from DPG regarding the number of plots, I will modify according to your proposal if this is the case. Otherwise, we can go with the current configuration now.
Hello @lmassacr , The change looks fine. How much will the output increase? @JianLIUhep , @botta (I actually don't know her git account, Jian, can you add her?), FYI. Chiara |
Hi @chiarazampolli, I ran the two QC configurations on one ctf file, and I obtain an increase of the output size by 10% with the new proposed configuration. Would this be acceptable? Cheers, |
Hello @lmassacr , I guess this is 10% of the MFT output, right? I don't recall what is the MFT total size, maybe @JianLIUhep does, from the exercise to reduce the size? In any case, it seems ok to me. Chiara |
Hi @chiarazampolli, This is a 10% increase of the outputs for the global muon tracks objects (which are additional objects on top of MCH, MID and MFT individual detector QCs). I can compare with the MFT output size (I have asked for their standalone workflows to run them, in case you want this number). I would expect that the global muon objects have sizes much smaller with respect to MFT outputs, because you have much less MCH+MID+MFT and MCH-MID matched tracks than the MFT standalone tracks, even if we duplicate some of the matched track types to apply different sets of cuts. Hope this clarify the number I circulated. Cheers, |
Change in the asynch QC config file for MFTMCHMID objects in order to apply different sets of cuts on MCH, MCH+MID tracks with respect to MCH+MFT or MCH+MID+MFT tracks.
A new variable to cut on the Chi^2 between MFT and MCH was introduced. It is defined in the TrackTasks.cxx class in QualityControl/Modules/MUON/Common/src (PR 2382)