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

Detsim signal simulation module. #734

Closed
wants to merge 3 commits into from

Conversation

gondiaz
Copy link
Collaborator

@gondiaz gondiaz commented Jul 24, 2020

Following issue #691:

This module contains the functions that compute the photoelectrons at each sensor using the LightTables and PSF. Also, the functions that generate the S1 times distribution.

This new version is the signal simulation module for detsim loop-based
implementation
@gondiaz
Copy link
Collaborator Author

gondiaz commented Sep 9, 2020

Added new signal simulation module for the loop-based implementation of detsim.

@gondiaz gondiaz mentioned this pull request Nov 19, 2020
@gondiaz
Copy link
Collaborator Author

gondiaz commented Nov 19, 2020

This PR is overriden by #756 #757.

@gondiaz gondiaz closed this Nov 19, 2020
carmenromo pushed a commit that referenced this pull request Feb 22, 2021
#757

[author: gondiaz]

This PR contains the detsim functions that simulate the S1 signal from the total
number of photons using the S1 LightTable. It is a single module `simulate_S1.py`
and its corresponding test module. Also, the S1 LightTable test file is added.

This PR, along with PR #756, overrides PRs #698, #699 and #734.

[reviewer: mmkekic]

This PR adds functionalities for simulation of the S1 scintillation light needed
for detsim city. The code is clear and tested, good job!
@gondiaz gondiaz deleted the detsim_simulate_signal branch September 9, 2021 07:12
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.

1 participant