Skip to content
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

Add t_pre and t_post to trun? #85

Open
mjtueros opened this issue Mar 20, 2024 · 3 comments
Open

Add t_pre and t_post to trun? #85

mjtueros opened this issue Mar 20, 2024 · 3 comments
Assignees
Labels

Comments

@mjtueros
Copy link
Contributor

currently we are saving t_pre and t_post, the parameters that define the time window arround the trigger time, in t_efieldsim.

I dont see where in the data files is this going to be stored. Should it be in trun?

@mjtueros
Copy link
Contributor Author

one aditional fact indicating this must be moved is that i gave to include efieldsim files in L1 level for DC2...this should not be needed if L1 level is equivalent to data.

@lwpiotr
Copy link
Contributor

lwpiotr commented Mar 29, 2024

In the hardware we have, or used to have (I need to get more info about new firmware) common_coincidence_time, pre_coincidence_window_ch, post_coincidence_window_ch. "There are three time windows which control the sample of ADC values to store and read out of the channel memory. Concatenated they enclose the total time frame of the event
data that is stored and read out.". If they are still coming from the hw, it would be best to adapt info from sims to them. Perhaps with setting common coincidence window to 0.

@mjtueros
Copy link
Contributor Author

well, i dont know about common_coincidence_time...might be the interval of time you allow for two to be in coincidence, but pre_coincidence_window_c and post_coincidence_window_ch sound very close to sims t_pre and t_post. I think we can add those to variables to trun now, and set the coomon_coincidence time to 0 as you suggest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants