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 dead-time information #75

Open
KrisThielemans opened this issue Dec 10, 2024 · 2 comments · May be fixed by #77
Open

add dead-time information #75

KrisThielemans opened this issue Dec 10, 2024 · 2 comments · May be fixed by #77
Assignees
Labels
enhancement New feature or request

Comments

@KrisThielemans
Copy link
Contributor

Suggestion after Data Elements meeting 29Nov2024:

Introduce new type of time-block with
i. Singles dead-time fraction encoded as 2D array (det_element, energy window)
ii. optional coincidence dead-time fraction encoded as 2D array (module1, module2)

Note that this needs a start/end time in each time block

@KrisThielemans KrisThielemans added the enhancement New feature or request label Dec 10, 2024
@KrisThielemans KrisThielemans self-assigned this Dec 10, 2024
@KrisThielemans
Copy link
Contributor Author

Actually, I think this needs to be formulated as "alive-time": the alive-time fraction (i.e. "detection probability") is computed as the product of the probabilities that both detectors are "alive" times the module-pair records coincidences (it is an "and" relation).

Example: suppose 2 detectors are only alive half the time, then the coincidence will be only alive a quarter of the time.

@KrisThielemans
Copy link
Contributor Author

Note that this needs a start/end time in each time block

We have a type for this already

TimeInterval: !record
fields:
start: uint # Type could maybe be RelativeTime in the future
stop: uint

KrisThielemans added a commit that referenced this issue Dec 12, 2024
Needed to change all time-blocks to use TimeInterval as opposed to
only giving the start time.

Fixes #75
@KrisThielemans KrisThielemans linked a pull request Dec 12, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant