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

Theta2 next-day results #89

Merged
merged 3 commits into from
Jan 24, 2022
Merged

Theta2 next-day results #89

merged 3 commits into from
Jan 24, 2022

Conversation

morcuended
Copy link
Member

@morcuended morcuended commented Jan 24, 2022

This PR adds several functions to produce theta2 from DL2 files based on https://github.com/cta-observatory/2022_01_lstchain_school/blob/main/DL1b_to_Dl2/Explore_DL2_data.ipynb

Plots are produced per source for each night and the results are uploaded to the lst1 internal webpage.

@codecov
Copy link

codecov bot commented Jan 24, 2022

Codecov Report

Merging #89 (057c2b6) into main (1d0a239) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   81.49%   81.40%   -0.10%     
==========================================
  Files          41       41              
  Lines        4307     4307              
==========================================
- Hits         3510     3506       -4     
- Misses        797      801       +4     
Impacted Files Coverage Δ
osa/utils/utils.py 67.13% <0.00%> (-1.41%) ⬇️
osa/utils/tests/test_utils.py 97.72% <0.00%> (-1.14%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d0a239...057c2b6. Read the comment docs.

@morcuended
Copy link
Member Author

I will merge this and test it. Probably we might want to use the post_dl2 analysis script from lstchain or even gammapy to produce the theta2 and signal significance quick results.

@morcuended morcuended merged commit 4a992ec into main Jan 24, 2022
@morcuended morcuended deleted the high-level branch January 24, 2022 19:34
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.

1 participant