-
Notifications
You must be signed in to change notification settings - Fork 30
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
fix: hide OMD, RPOTS, LOWQ2 errors for single particle simulations #1716
base: main
Are you sure you want to change the base?
Conversation
FYI @rahmans1 You can likely also benefit from this for singles productions. |
Not sure why it is setting parameters incorrectly, see https://github.com/eic/EICrecon/actions/runs/12880836546/job/35910542823?pr=1716#step:8:4553 |
Same was already the case for jana options, https://github.com/eic/EICrecon/actions/runs/12880836546/job/35910542823?pr=1716#step:8:4713. |
|
Capybara summary for PR 1716
|
Ignoring iwyu suggestion of |
Briefly, what does this PR introduce?
This removes the endless
No beam protons to choose matrix!! Skipping!!
warnings which are clogging up the logs and hiding real issues. Do the same with the warnings about missing beam electrons.What kind of change does this PR introduce?
Please check if this PR fulfills the following:
Does this PR introduce breaking changes? What changes might users need to make to their code?
No.
Does this PR change default behavior?
No.