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

DM-12851: Determine whether dataset framework can be made generic #16

Merged
merged 1 commit into from Jan 3, 2018

Conversation

kfindeisen
Copy link
Member

This PR updates the documentation of the dataset framework to remove unnecessary requirements from the dataset format.

Ingestion does not impose any constraints like an
observatory-specific directory structure.
@kfindeisen kfindeisen requested a review from mrawls January 3, 2018 00:54
.. TODO: are these more standardized? are they available from somewhere? (DM-12851)

* The :file:`refcats` directory contains one or more tar'd HTM files containing astrometric and photometric reference catalogs.
The directories may have any internal structure.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please clarify whether this point applies only to raw and calib or to all the directories mentioned in this section.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it confusing only because of the diff? This is how it looks in HTML:

  • The raw and calib directories contain science and calibration data, respectively. The directories may have any internal organization.
  • The templates directory contains a LSST Butler repository containing processed images useable as templates. Template files must be TemplateCoadd files produced by a compatible version of the LSST science pipeline.
  • The refcats directory contains one or more tar files, each containing containing one or more astrometric or photometric reference catalogs in HTM shard format.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, yes, the diff made it look like its own bullet point at a glance. All good!

@kfindeisen kfindeisen merged commit f5a2481 into master Jan 3, 2018
@kfindeisen kfindeisen deleted the tickets/DM-12851 branch November 30, 2018 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants