Skip to content
This repository was archived by the owner on Mar 31, 2021. It is now read-only.

Hipehdf5#21

Open
devine73 wants to merge 3 commits into
cta-observatory:masterfrom
devine73:hipehdf5
Open

Hipehdf5#21
devine73 wants to merge 3 commits into
cta-observatory:masterfrom
devine73:hipehdf5

Conversation

@devine73

Copy link
Copy Markdown

As announced in the ASWG meeting on Thursday 9th, the HDF5 reader for the HDF5 data format from the LAPP for R1 (direct conversion from simtel file). Add the proper test file gamma_test_large.simtel.gz converted into HDF5 gamma_test_large.h5
To contact the author:
pierre.aubert@lapp.in2p3.fr

@maxnoe

maxnoe commented May 10, 2019

Copy link
Copy Markdown
Member

Could you rename the file that it is clear that this is in this hipecta format? eg. gamma_test_prod3_hipecta.hdf5?

@devine73

Copy link
Copy Markdown
Author

So, is it ok now ? (I updated also the tests in ctapipe with the proper file name).

@kosack

kosack commented May 10, 2019

Copy link
Copy Markdown
Member

I think this may blow the size limit of the package... This reminds me that we need to address the issue cta-observatory/ctapipe#738 soon, since ctapipe-extra has become too large.

@kosack

kosack commented May 10, 2019

Copy link
Copy Markdown
Member

How do the tests of e.g. ctapipe_io_lst, etc work? I don't think they all have example files in ctapipe-extra?

@kosack

kosack commented May 16, 2019

Copy link
Copy Markdown
Member

see cta-observatory/ctapipe#1073 -> we now have space for larger test datasets.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants