Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

Allow instructions with 0 amplitude #373

Open
terliuk opened this issue May 20, 2022 · 0 comments
Open

Allow instructions with 0 amplitude #373

terliuk opened this issue May 20, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@terliuk
Copy link
Contributor

terliuk commented May 20, 2022

I think that we need to allow instructions with 0 amplitude rather than rejecting all of them , as it is possible that energy deposit produces 0 electrons or photons , but we still want to keep track about it in truth.

Here is the corresponding line:
https://github.com/XENONnT/WFSim/blob/master/wfsim/strax_interface.py#L617-L618
@terliuk terliuk added the enhancement New feature or request label May 20, 2022
@JoranAngevaare JoranAngevaare mentioned this issue Jul 18, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@terliuk