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

Landing page for mask bits documentation #118

Closed
weaverba137 opened this issue Jun 7, 2022 · 5 comments
Closed

Landing page for mask bits documentation #118

weaverba137 opened this issue Jun 7, 2022 · 5 comments
Assignees

Comments

@weaverba137
Copy link
Member

We need a central "landing page" for documentation of all mask bits, analogous to SDSS's bitmask page.

It is sufficient to link to mask bits documentation that is stored elsewhere. For example, bits used by DECaLS can continue to be described on https://www.legacysurvey.org, but the new landing page should at least link to that documentation.

As a stretch goal, we could also describe other integers used in DESI, such as TARGETID, even though it is not technically a mask.

We are not advocating moving mask bits in code at this time. In other words, this issue is not intended to implement desihub/desiutil#113.

@julienguy
Copy link
Contributor

Can @weaverba137 provide us with a placeholder file for the bitmask and an example for how link to it in the .rst files ?

@weaverba137
Copy link
Member Author

Is the documentation here insufficient: https://desidatamodel.readthedocs.io/en/latest/datamodel.html

I think everyone would be much better served if I improved the documentation on that page so that you could provide this yourself.

Or am I missing something about the question?

@julienguy
Copy link
Contributor

The documentation is good (I managed to cross-link the fibermap doc for instance). Could we have a placeholder page for bitmask in the repo so we can start writing correct links?

@weaverba137
Copy link
Member Author

Glad the documentation is good. You can start from this file: https://github.com/desihub/desidatamodel/blob/main/doc/bitmasks.rst. Branch of off main to edit it.

@weaverba137
Copy link
Member Author

This page was created by #139.

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

No branches or pull requests

3 participants