-
Notifications
You must be signed in to change notification settings - Fork 97
Optionally produce the EWM as well as the event header #1693
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi @michaelmackenzie,
which require these tests: build. @Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main. ⌛ The following tests have been triggered for df426c1: build (Build queue - API unavailable) |
|
☔ The build tests failed for df426c1.
N.B. These results were obtained from a build of this Pull Request at df426c1 after being merged into the base branch at 440d6ec. For more information, please check the job page here. |
|
@FNALbuild run build test with Mu2e/mu2e-trig-config#134 |
|
⌛ The following tests have been triggered for df426c1: build (Build queue - API unavailable) |
|
☀️ The build tests passed at df426c1.
N.B. These results were obtained from a build of this Pull Request at df426c1 after being merged into the base branch at 440d6ec. For more information, please check the job page here. |
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.
The tick unit is 25ns in the CFO event duration - so should be revised!
rrivera747
left a comment
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.
tick should be 25ns
|
@FNALbuild run build test with Mu2e/mu2e-trig-config#134 |
|
⌛ The following tests have been triggered for 03385a1: build (Build queue - API unavailable) |
|
☀️ The build tests passed at 03385a1.
N.B. These results were obtained from a build of this Pull Request at 03385a1 after being merged into the base branch at cdbcae2. For more information, please check the job page here. |

This allows for the creation of the
EventWindowMarkerwhen creating theEventHeaderobject Online. There are also a few minor edits to theEventHeadercreation logic. TheEventHeaderdefinition should be reviewed in a future discussion, to ensure we have the necessary data in it without unnecessarily increasing its size (which directly impacts the data volume of the luminosity stream).