-
Notifications
You must be signed in to change notification settings - Fork 83
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
Ext mon detector restructure #1325
base: main
Are you sure you want to change the base?
Conversation
Resolve conflicts in rebase
Resolve conflict in txt file rebase
Simplify scintillator implementation
Hi @fine1-sam,
which require these tests: build. @Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main. The following users requested to be notified about changes to these packages: ⌛ The following tests have been triggered for fa6a31b: build (Build queue is empty) |
To clarify: there are no overlaps with the default settings, and I will want to merge this work after a review. |
☔ The build tests failed for fa6a31b.
N.B. These results were obtained from a build of this Pull Request at fa6a31b after being merged into the base branch at 01eb5be. For more information, please check the job page here. |
I marked this as a draft since it's failing the CI tests. @gaponenko What are the plans to move forward? |
📝 The HEAD of |
Work in progress
There are overlapping geometries in the ext mon detector modules when the stackRotation flag is set to "true."