-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add minimum and maximum values for theta2 and gammaness cuts. #57
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks fine, thanks for implementing this
Hi, just for reference, I post here the cuts before and after introducing these min/max values. Before
and after:
|
Hi @aleberti, thanks for the pull request. It looks fine, but I have a question - are the min/max values the ones used in the MARS analysis? It looks that setting the min/max values only change the following theta cut (set to the min value) and so the spectrum does not change between before and after the changes?
|
yes, indeed the SED did not change with these cuts. Just for testing, I modified locally
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, thanks for testing. I will merge this to the master now.
This PR adds minimum and maximum values for the theta2 and gammaness cuts.