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
Document the Data Model for MTL #101
Conversation
… an empty ops index page; to be filled later.
This should address #97. |
In case it's helpful for reviewers, I've dumped the sphinx output here: https://data.desi.lbl.gov/desi/users/adamyers/ADM-dm-MTL/html/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine to me. A couple of minor questions that should be addressed before merging.
For the record, we currently don't have any support for validating .ecsv files, but it's something we need to do anyway and this PR provides much stronger motivation for that.
Related: ECSV files don't have the concept of multiple HDUs, at least in versions I've looked at, so in the future we might remove the Contents section. But let's not worry about that for this PR.
Thanx for the changes. OK to merge when ready. |
This PR documents the data model for the MTL ledgers.
As part of this PR, a new
$SURVEYOPS
directory at the root level of the data model is added to document files used by survey operations.