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

New production 20241120_v0.10.12_src13_dec_min_1802_tuned #512

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

Conversation

gaia-verna
Copy link

@gaia-verna gaia-verna commented Nov 20, 2024

New Prod config

Self check-list:

  • I have checked the lstchain config, in particular for:
    • az_tel instead of sin_az_tel if data to be analyzed have been produced with lstchain <= v0.9.7
    • "increase_nsb" and "increase_psf" are provided in "image_modifier" (if used)
  • I have checked the environment in the lstmcpipe config and it is the one used to analyse DL>1 data
  • I have provided the command (in README), or script (in additionnal .py file) used to produce the lstmcpipe config

PROD 20241120_v0.10.12_src13_dec_min_1802_tuned

Contact: Gaia V.

Short description of the config

'''
lstmcpipe_generate_config PathConfigAllSkyFullDL1ab --dec_list dec_min_1802 --prod_id 20241120_v0.10.12_src13_dec_min_1802_tuned --kwargs source_prod_id=20241120_v0.10.12_src13_dec_min_1802_tuned
'''

Config for the NSB tuning:

"image_modifier": {
"increase_nsb": true,
"extra_noise_in_dim_pixels": 2.335,
"extra_bias_in_dim_pixels": 0.785,
"transition_charge": 8,
"extra_noise_in_bright_pixels": 2.985
}

Why this config is needed

I need this production to analyse the LST-1 data of src13 (see https://www.lst1.iac.es/wiki/index.php/MC_analysis_and_IRF_production#lstmcpipe_productions) with dec -18.02 deg. The median of the NSB standard deviation for this dataset is approximately 1.940 p.e.

@gaia-verna
Copy link
Author

Hi @vuillaut, can I ask you for the review of this PR? Thanks!

@vuillaut
Copy link
Member

vuillaut commented Dec 2, 2024

Hi @gaia-verna
Could you check what would be the nsb_tuning_ratio at waveform level using https://github.com/cta-observatory/cta-lstchain/blob/main/lstchain/scripts/lstchain_tune_nsb_waveform.py please ?
As we are moving to the new scheme with a grid of NSB-tuned MC, the models and IRFs you need are probably already produced, and the tuning will be better since it's now done at waveform level.
See presentations from https://indico.cta-observatory.org/event/5741/timetable/#20241112.detailed if needed.

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.

2 participants