Skip to content

Switch to unix_ts_usec for event t0#22

Merged
mjkramer merged 1 commit into
developfrom
mjkramer/feature/trigger_times
Jul 29, 2025
Merged

Switch to unix_ts_usec for event t0#22
mjkramer merged 1 commit into
developfrom
mjkramer/feature/trigger_times

Conversation

@mjkramer
Copy link
Copy Markdown
Collaborator

Paired with DUNE/ndlar_flow#195 which adds the unix_ts_usec field to charge/events. Using it improves the trigger matching between 2x2 and Mx2 in the CAFmaker.

@mjkramer mjkramer merged commit 3095594 into develop Jul 29, 2025
@sindhu-ku
Copy link
Copy Markdown
Collaborator

Is this the same for FSD, and the same for sim and data?
If not maybe we should add an if else based on config?

@mjkramer
Copy link
Copy Markdown
Collaborator Author

For FSD, and for 2x2 sim, flow just fills unix_ts_usec with the old ts_start field (rescaled from ticks to microseconds). That's configured by the pps_delay_corrector_enabled param of flow's event builder. So no need for extra config or logic in flow2supera!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants