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

add desimodel files to datamodel #6

Closed
sbailey opened this issue Nov 16, 2015 · 5 comments · Fixed by #31
Closed

add desimodel files to datamodel #6

sbailey opened this issue Nov 16, 2015 · 5 comments · Fixed by #31
Assignees

Comments

@sbailey
Copy link
Contributor

sbailey commented Nov 16, 2015

The desimodel files are lightly documented in the desimodel/doc directory and in DESI-0847, but we should add more detailed data model documentation to desidatamodel. This would be a new top-level DESIMODEL directory, at the same level as DESI_SPECTRO_REDUX, etc.

@weaverba137
Copy link
Member

In light of #7, we should really get these data models written.

@weaverba137
Copy link
Member

The structure exists for this now, but no actual content has been added yet.

@weaverba137 weaverba137 self-assigned this Apr 25, 2017
@weaverba137
Copy link
Member

To support the 2017 Data Challenge, I'd like to get this done.

@weaverba137 weaverba137 added this to the 2017 Data Challenge milestone Apr 25, 2017
@weaverba137
Copy link
Member

I'm starting to add files on the desimodel-update branch. Feel free to take a look at those and edit as needed.

@weaverba137
Copy link
Member

At this point, I've added files that describe all areas of the desimodel data directory, except the spectra directory. While examining that directory, a few other questions came up that need to be answered before I can proceed (see desihub/desimodel#47, desihub/desimodel#48, desihub/desimodel#49) . In the meantime, you can look at and further edit the files on the desimodel-update branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants