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
Raw data updates #151
Raw data updates #151
Conversation
@sbailey, I think you could start by reviewing |
* main: Update change log adding type to WEIGHT clustering columns Adding more columns related to LSS. Remove some unused columns that had no description, and solve a minor bug in PROB_OBS column Adding last set of columns for the description of the LSS datamodel
* main: Add descriptions to SkyCam HDUs
* main: Update guide-EXPID.rst
@sbailey, I'd like to merge this soon. I think I've got about as much information into the raw data model files as we're going to get. |
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.
Sounds good. Merge when ready. I haven't reviewed every file in detail, but this is definitely a step forwards compared to current main, so let's merge and make any additional updates in future PRs if needed. Thanks.
This PR primarily updates the data model for the raw data files,
desi-EXPID.rst
. Remaining items before merging:SPEC
HDU.The
desi-EXPID.rst
file can probably be reviewed as-is, and other files primarily have structural changes.The changes to the code allow the data model verification script to select multiple candidate files for comparison, rather than just the first one it finds.
A few new data model files have been added to reduce the number of extraneous files detected in the raw data.