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

Pass allowed_tels down to eventio in simtel event source #987

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

maxnoe
Copy link
Member

@maxnoe maxnoe commented Feb 27, 2019

Passes the allowed_tels down to the SimTelFile, so it get's a little faster when reading only a few telescopes.

@maxnoe maxnoe requested review from dneise and kbruegge February 27, 2019 14:04
@maxnoe maxnoe changed the title Pass allowed_tels to simtel event source Pass allowed_tels down to eventio in simtel event source Feb 27, 2019
@codecov
Copy link

codecov bot commented Feb 27, 2019

Codecov Report

Merging #987 into master will increase coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #987      +/-   ##
==========================================
+ Coverage   78.19%   78.35%   +0.15%     
==========================================
  Files         194      194              
  Lines       11313    11313              
==========================================
+ Hits         8846     8864      +18     
+ Misses       2467     2449      -18
Impacted Files Coverage Δ
ctapipe/io/simteleventsource.py 100% <ø> (ø) ⬆️
ctapipe/tools/camdemo.py 76.13% <0%> (+20.45%) ⬆️

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 dcb64d4...9a7a1b2. Read the comment docs.

@maxnoe maxnoe merged commit 7a40dba into master Feb 27, 2019
@maxnoe maxnoe deleted the allowed_tels branch February 27, 2019 18:09
kosack added a commit that referenced this pull request Mar 8, 2019
* removed PIP installs, and use conda instead

* updated all env files

* added back pyhessio, needed for tests

* Add missing imports for event sources (#991)

* Pass allowed_tels to simtel event source (#987)

* removed PIP installs, and use conda instead

* updated all env files

* added back pyhessio, needed for tests

* removed pyhessio from environment.yml (users don't need it, just tests)
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.

3 participants