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

Update r0 sample start and end #354

Merged

Conversation

pawel21
Copy link
Collaborator

@pawel21 pawel21 commented Apr 20, 2020

Hi,

I have changed default value of start and end sample in waveform in 'r0.py' and updated notebook according to the change in EVB in November 2019.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@rlopezcoto
Copy link
Contributor

This PR follows from this discussion by e-mail by @jsitarek:

During the call today we discussed the removal of slices and if it can affect the pulse shape correction.
Before the firmware upgrade the first 2 and last 2 samples had some extra noise and had to be removed from the analysis. After the upgrade it shifted by one and now first 3 and last 1 have to be removed. Currently in the code still 2+2 are removed.
When you use the method of Taka on the R1 files you should be aware of this shift, so e.g. if you skip the first 2 samples, and first capacitor is 111 then in fact the first sample that is recorded is 111+2 = 113. If you make the tables from files that have this skipping done and apply it to a file with the same format it might even work fine, but if you apply to files without skipping, or with different number of skipped samples this can completely screw up the method.

@codecov
Copy link

codecov bot commented Apr 20, 2020

Codecov Report

Merging #354 into master will increase coverage by 0.18%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #354      +/-   ##
==========================================
+ Coverage   43.52%   43.70%   +0.18%     
==========================================
  Files          69       69              
  Lines        4717     4791      +74     
==========================================
+ Hits         2053     2094      +41     
- Misses       2664     2697      +33     
Impacted Files Coverage Δ
lstchain/calib/camera/r0.py 13.88% <100.00%> (ø)
lstchain/reco/r0_to_dl1.py 66.04% <0.00%> (-3.19%) ⬇️

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 686c702...1bc55f7. Read the comment docs.

@rlopezcoto rlopezcoto merged commit c9a7849 into cta-observatory:master Apr 20, 2020
@pawel21 pawel21 deleted the update_r0_sample_start_and_end branch July 10, 2020 11:22
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.

2 participants