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-11597: Integration in pipelines.lsst.io docs #16
Conversation
The package documentation system no longer relies on README and requirements.txt to setup an ad hoc Sphinx build.
This is the templated boilerplate for package documentation. The manifest declares lsst.verify **module** documentation and verify **package** documentation directories.
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.
I gave a very cursory copy edit pass. But looks good overall.
python/lsst/verify/jobmetadata.py
Outdated
Parameters | ||
---------- | ||
data : `dict`-like | ||
The ``data`` object need to provide an ``items`` method it |
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.
need -> needs
it -> to ?
python/lsst/verify/measurement.py
Outdated
``extras`` work just like `Blob`\ s, but they're automatically created with | ||
each `Measurement`. Add `~lsst.verify.Datum`\ s to ``extras`` if those | ||
`~lsst.verify.Datum`\ s only make sense in the context of that | ||
`Measurement`. If `Datums`\ s are relevant to multiple measurement, add |
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.
Datums -> Datum
python/lsst/verify/measurement.py
Outdated
|
||
These key-value pairs are automatically available from `Job.meta`, | ||
though keys are prefixed with the `Metric`\ 's name. This metadata can | ||
be queried by `Specification`\ s, so that `Specification` \s can be |
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.
\s -> \ s
python/lsst/verify/measurementset.py
Outdated
@@ -30,7 +30,11 @@ | |||
|
|||
|
|||
class MeasurementSet(JsonSerializationMixin): | |||
"""A collection of measurements of metrics. | |||
"""A collection of `~lsst.verify.Measurement` s of |
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.
do you need the escaping ?
python/lsst/verify/metric.py
Outdated
Units of the metric. `Measurements` of this metric must be in an | ||
equivalent (i.e. convertable) unit. Argument can either be a | ||
`~astropy.unit.Unit` instance, or a an astropy.unit.Unit-compatible | ||
Units of the metric. `~lsst.verify.Measurements` of this metric must be |
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.
Measurements -> Measurement
\ s?
python/lsst/verify/specset.py
Outdated
exist in this set are added to this one. Specification in ``other`` | ||
replace specifications of the same name in this one. | ||
Another `SpecificationSet`. `Specification`\ s in ``other`` that do | ||
exist in this set are added to this one. `Specification`\ s in |
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.
do -> do not
This is a sweep through the docstrings to fix obvious errors and typos. The fixes are mostly to add missing docstrings, improve code links, and fix formatting issues.
a9da19c
to
6c5e70d
Compare
This PR integrates
verify
andlsst.verify
into the pipelines.lsst.io documentation build.validate_base
(alas, new user guides need to be written forlsst.verify
still).Homepage for
verify
package: https://pipelines.lsst.io/v/DM-11597/packages/verify/index.htmlHomepage for
lsst.verify
module: https://pipelines.lsst.io/v/DM-11597/modules/lsst.verify/index.html