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

Set correct NSB tuning settings for low-zenith Crab MC #1025

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

moralejo
Copy link
Collaborator

NSB parameters are obtained with the fixed code for finding the NSB-tuning parameters (#1024). Note: the replaced parameters were very outdated, not obtained with the immediately earlier version, and were not those used for Crab in the analyses over the past few months.

Also switched off obsolete optical PSF tuning.

NSB parameters are obtained with the fixed code for finding the NSB-tuning parameters (#1024). 
Note: the replaced parameters were very outdated, not obtained with the immediately earlier version, and were not those used for Crab in the analyses over the past few months.

Also switched off obsolete optical PSF tuning.
@codecov
Copy link

codecov bot commented Oct 19, 2022

Codecov Report

Base: 71.62% // Head: 71.62% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (33188df) compared to base (089f641).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1025      +/-   ##
==========================================
- Coverage   71.62%   71.62%   -0.01%     
==========================================
  Files         120      120              
  Lines       11302    11301       -1     
==========================================
- Hits         8095     8094       -1     
  Misses       3207     3207              
Impacted Files Coverage Δ
lstchain/image/modifier.py 88.39% <0.00%> (-0.07%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jsitarek
Copy link
Collaborator

looks fine to me.
Just for the reference, here are the values that I am using for the LST1+MAGIC analysis: noise_dim: 1.56, bias_dim: 0.52, transition: 8, noise_bright: 1.8.
They are quite similar

@moralejo moralejo merged commit 50e7d0d into master Oct 19, 2022
@moralejo moralejo deleted the moralejo-patch-2 branch October 19, 2022 14:55
@moralejo moralejo changed the title Set correct NSB tuning settings for low-zenith MC Set correct NSB tuning settings for low-zenith Crab MC Oct 27, 2022
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.

3 participants