oirlab / liger_iris_pipeline

Data Reduction System (DRS) for the Thirty Meter Telescope IRIS imager/spectrograph
https://oirlab.github.io/iris-pipeline
0 stars 3 forks source link

Documentation about file formats and schemas #23

Closed zonca closed 4 years ago

zonca commented 4 years ago

@arunsurya77 @ikashell started writing docs about file format,

@arunsurya77 can you please add link to the document you prepared? even if not accessible publicly is handy for us

arunsurya77 commented 4 years ago

Is this regarding the file formats. I am adding the link to the file formats document. https://docs.google.com/document/d/1ppkyuNsjVyTtxLoCQM04nfjET_ZmqywTiw9LiR-ny78/edit?usp=sharing

On Thu, 7 May 2020 at 13:09, Andrea Zonca notifications@github.com wrote:

@arunsurya77 https://github.com/arunsurya77 @ikashell https://github.com/ikashell started writing docs about file format,

@arunsurya77 https://github.com/arunsurya77 can you please add link to the document you prepared? even if not accessible publicly is handy for us

You can view, comment on, or merge this pull request online at:

https://github.com/oirlab/iris_pipeline/pull/23 Commit Summary

  • docs: file formats and schemas

File Changes

Patch Links:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/oirlab/iris_pipeline/pull/23, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKB37LVA3OTBIYRYHLWGU73RQMIP7ANCNFSM4M3T4UGA .

zonca commented 4 years ago

from @ikashell:

will this be merged with Arun's document on extensions/units, etc? i think in terms of schemas this is good we might say something about how schemas will be handled for raw readout --> raw science frame processing?

zonca commented 4 years ago

ok, @arunsurya77 's document linked above in this thread has been updated to include the modification to the file formats required to make it compatible with iris_pipeline. handling raw readouts is not implemented yet in iris_pipeline so I would wait before adding docs about that.

I think for now I can merge this, review probably once we have #7 in good shape.