Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
weighting MC gamma to MAGIC log-parabola spectrum #302
weighting MC gamma to MAGIC log-parabola spectrum #302
Changes from 11 commits
2ac0d06
c200880
99cb79e
3c8a6af
368d973
e9071c5
35a67b0
b2c3ab7
66dd663
63f0647
29b10ba
5867370
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Hi @SeiyaNozaki, I'm happy with the changes and would propose this to be accepted if @moralejo approves, just one additional question: if the
shape
given isLogParabola
and nobeta
parameter is given, will the call throw an error or is there any default value?Also, what if
shape == PowerLaw
and I introduce abeta
parameter in the call?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.
Hi @rlopezcoto ,
For the first case(
LogParabola
withoutbeta
), we will get error.But the second case(
PowerLaw
withbeta
), no error and just other parameters are used for the calcurlation.Would it be better to add the function to check if the
param
match theshape
?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.
what kind of error in the first case? (a generic python one I guess).
It would be great if you could check that we are giving the correct parameters depending on the function introduced.
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.
Yes, just a KeyError.
I'll introduce such a function in a few days.
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.
Same as above, explain w_param, especially signs.