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

WIP: Update lstchain to v096... #2

Open
wants to merge 2,137 commits into
base: lstchain_io_RTA
Choose a base branch
from

Conversation

garciagenrique
Copy link
Owner

@garciagenrique garciagenrique commented Aug 2, 2022

...onto the lstchain_io_RTA branch.

Full changelog between lstchain-v0.6.3 and lstchain_io_RTA branches:
cta-observatory/cta-lstchain@master...garciagenrique:cta-lstchain:v063_RTAv042_ped_ugly_fix

Changes:

Important Notes

  • ctapipe_io_mchdf5 won't be any longer working (ctapipe=0.12 has evolved a lot). Nevertheless, we should start using the simtel --> hdf5 conversion provided by ctapipe >=v0.13.0 link. This means that:
    • The new env won't be able to make the conversion until lstchain gets updated (v0.9.7?) and incorporates ctapipe v0.15.0 Work in progress.
    • However, this update depends on ctapipe_io_lst and some dependencies updates WIP here.
    • This will take some time so until this happens and we can use the ctapipe R0-HDF5 conversion, ctapipe_io_mchdf5 will not be installed in the new RTA environment (thus, to create again the HDF5 files, if ever, an old env should be used).

chaimain and others added 30 commits January 31, 2022 09:55
…ted to cuts. And also addressed other issues raised
…p_pytest

Add pytest for source-dependent analysis
vuillaut and others added 30 commits April 8, 2022 09:57
Allow selecting N samples for training and testing
…l3_cuts

Remove hard-coded arguments in the energy-dependent cuts functions
Reduce size of random forests in standard config
…e/cta-lstchain into addapt_rta_dl2dl23_scripts
Addapt RTA dl2-dl3 script and tools into RTA branch
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.

9 participants