-
Notifications
You must be signed in to change notification settings - Fork 9
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
Incorrect Trigger type CFCAL #791
Labels
enhancement
New feature or request
Comments
@somovalex Should this be included before we start REST production? |
for the record, this code is called in the data reconstruction to estimate the trigger calorimeter energy sums, and the trigger bit info is ignored. So maybe the short term fix is to limit the number of times this error message is printed out. Of course, simulation of this trigger type is needed for acceptance corrections. |
Yes, this would be the temporary solution (before I finish up with the final implementation)
…________________________________
From: Sean Dobbs ***@***.***>
Sent: Thursday, March 28, 2024 5:34 PM
To: JeffersonLab/halld_recon ***@***.***>
Cc: Alexander Somov ***@***.***>; Mention ***@***.***>
Subject: [EXTERNAL] Re: [JeffersonLab/halld_recon] Incorrect Trigger type CFCAL (Issue #791)
for the record, this code is called in the data reconstruction to estimate the trigger calorimeter energy sums, and the trigger bit info is ignored. So maybe the short term fix is to limit the number of times this error message is printed out.
Of course, simulation of this trigger type is needed for acceptance corrections.
—
Reply to this email directly, view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_halld-5Frecon_issues_791-23issuecomment-2D2026170169&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=UEJkvUcBvE4mBa6l1pQs5Q&m=96R7jM-DgCf78QQy78e_DaCEMiPtUnZcw8tuwX9kmvvAlTMrMVgEMUFF9gWRMFcB&s=QhcUAxTOnlcxyxPP5xhfO46999bazUIpxNVtWb54XqU&e=>, or unsubscribe<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AD4KOG2LCLKTZVFWICT3U6TY2SEHLAVCNFSM6AAAAABFHWM35CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGE3TAMJWHE&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=UEJkvUcBvE4mBa6l1pQs5Q&m=96R7jM-DgCf78QQy78e_DaCEMiPtUnZcw8tuwX9kmvvAlTMrMVgEMUFF9gWRMFcB&s=Kec7E-TJp6kMi8ZcGJ-32VHJNKenFiga1_SnGW8bthc&e=>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For the 2022-08 PrimEx data set, the current halld_recon master prints many error messages of this type:
The trigger emulation does not seem to include this type in
halld_recon/src/libraries/TRIGGER/DL1MCTrigger_factory.cc
Line 891 in 1d98aa5
The text was updated successfully, but these errors were encountered: