-
Notifications
You must be signed in to change notification settings - Fork 77
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 MAGIC sensitivity with references #567
Conversation
- CRAB spectrum from pyIRF - fix a few formatting issues - add unit tests on changed functions
lstchain/mc/plot_utils.py
Outdated
@@ -331,23 +336,22 @@ def plot_positions_survived_events(df_gammas, | |||
""" | |||
|
|||
e_reco_g = 10**df_gammas.mc_energy | |||
e_reco_p = 10**df_protons.mc_energy | |||
e_reco_p = 10 ** df_protons.mc_energy |
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.
Wat?
ereco = 10**mc_energy ?
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 did not write that code but good catch.
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.
This had nothing to do with this PR but I updated it nonetheless to take the reco energy from the dataframe
Codecov Report
@@ Coverage Diff @@
## master #567 +/- ##
==========================================
+ Coverage 50.43% 50.96% +0.53%
==========================================
Files 84 85 +1
Lines 7317 7335 +18
==========================================
+ Hits 3690 3738 +48
+ Misses 3627 3597 -30
Continue to review full report at Codecov.
|
This is stalled, but still needed, right? - @vuillaut, can you solve the conflicts? |
Done |
Merged in #622 |
Following the discussions about having a proper reference for MAGIC sensitivity, here is a PR fixing that.
The result should look like this: