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

Update of the single photoelectron response file #1238

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Mitsunari-Takahashi
Copy link

The single photoelectron spectrum data lstchain/data/SinglePhE_ResponseInPhE_expo2Gaus.dat has an artificial bump in the spectrum due to the analysis method, and we made a new one lstchain/data/spe_LST1_2022-04-27_AP2.0e-4.dat to remove it. In addition, the new spectrum is folded with the afterpulsing while the old one is not.

Please find the details at cta-observatory/lst-sim-config#40.

@rlopezcoto
Copy link
Contributor

thanks @Mitsunari-Takahashi !
As I understood from the discussion in cta-observatory/lst-sim-config#40, this will be the file used for Prod6. This change will mainly be affecting the NSB tuning here in lstchain, are there any changes that need to be applied in ctapipe_io_lst as well for the calibration @FrancaCassol , @maxnoe ?

@maxnoe
Copy link
Member

maxnoe commented Feb 27, 2024

Is this the single pe pulse shape? Or the single pe integration spectrum?

ctapipe_io_lst only uses the single pe pulse shape (to fill it into the SubarrayDescription / CameraReadout ).

The file used currently is

oversampled_pulse_LST_8dynode_pix6_20200204.dat

here: https://github.com/cta-observatory/ctapipe_io_lst/blob/main/src/ctapipe_io_lst/resources/oversampled_pulse_LST_8dynode_pix6_20200204.dat

@Mitsunari-Takahashi
Copy link
Author

Is this the single pe pulse shape? Or the single pe integration spectrum?

It's the LST1-PMT output charge spectrum integrated over an extraction time window.

I missed the format of the new dat file different from the old one. So please reject this pull request. Sorry, I will correct the file. Before that, could anyone confirm whether the NSB tuning needs the AP-folded spectrum?

@moralejo
Copy link
Collaborator

Is this the single pe pulse shape? Or the single pe integration spectrum?

It's the LST1-PMT output charge spectrum integrated over an extraction time window.

I missed the format of the new dat file different from the old one. So please reject this pull request. Sorry, I will correct the file. Before that, could anyone confirm whether the NSB tuning needs the AP-folded spectrum?

NSB tuning at the waveform level (which is the preferred method) needs both the pulse template (shape) and the amplitude fluctuations. About the APs, I always find it quite confusing to mix them with the spectrum of amplitude fluctuations. It is ok for the NSB, because it is anyway arriving at random times. But for the p.e.'s from C-photons we should never use such a spectrum of amplitude fluctuations including APs, right? The APs produced by p.e.'s from C-photons should not be part of the same event, but produce a delayed signal.

So my understanding is that for the specific case of simulating NSB is it fine (even convenient) to include the APs, but not for the simulation of the signal (which of course it is not done in lstchain!). But I am not 100% sure.

@Mitsunari-Takahashi
Copy link
Author

So my understanding is that for the specific case of simulating NSB is it fine (even convenient) to include the APs, but not for the simulation of the signal (which of course it is not done in lstchain!). But I am not 100% sure.

I see, so we should separate these problems.

For lstchain, the single p.e. spectrum should be folded by AP because it is used only for the NSB tuning (See the diff list in the commits). I will make a new pull request.

In addition, to implement the NSB tuning at the waveform level, I understand we need the pulse shape template. I'd like to remark that the AP pulse is shorter than the photon pulse, but probably the difference is negligible.

For the Cherenkov photon signal simulation, I agree that the AP should not be folded. Is this a matter of lst-sim-config? @moralejo

@moralejo
Copy link
Collaborator

For lstchain, the single p.e. spectrum should be folded by AP because it is used only for the NSB tuning (See the diff list in the commits). I will make a new pull request.

Correct

In addition, to implement the NSB tuning at the waveform level, I understand we need the pulse shape template. I'd like to remark that the AP pulse is shorter than the photon pulse, but probably the difference is negligible.

I'd say so... APs are after all just a small part of the noise.

For the Cherenkov photon signal simulation, I agree that the AP should not be folded. Is this a matter of lst-sim-config? @moralejo

I just checked that it is handled properly in the simulation (I had no doubts). From the sim_telarray documentation:
"The s.p.e. response tables also accounts for afterpulses. Since afterpulses from the Cherenkov light itself arrives much too late to be registered with the shower, afterpulses are only taken into account for night-sky background light, where it doesn’t matter if the original photo-electron was a few hundred nanoseconds earlier"

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.

4 participants