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-16535: Implement MetricRegistry #33
Conversation
dcfc87a
to
960a3b3
Compare
A variety of questions, most of which can probably be cleared up with revised documentation or added inline-comments in the code. Exactly how I hope you can move your Yay deleting code! |
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.
A variety of questions/clarifications requested.
doc/lsst.verify/tasks/lsst.verify.compatibility.MetricsControllerTask.rst
Outdated
Show resolved
Hide resolved
doc/lsst.verify/tasks/lsst.verify.compatibility.MetricsControllerTask.rst
Outdated
Show resolved
Hide resolved
This commit removes most of the hardcoding from MetricsControllerTask.
This class was added in DM-16642, but I declined to document it because I thought it shouldn't be emphasized to the user. However, the framework for documenting subtasks (in this case, of MetricsControllerTask) requires that each default subtask have a task topic page and not just an API entry.
960a3b3
to
d851d09
Compare
d851d09
to
e293ec9
Compare
This PR adds infrastructure to
lsst.verify.compatibility
for configuringMetricTask
s, and modifiesMetricsControllerTask
to use it. Since this change makesMetricsControllerTask
usable outside of carefully controlled environments, this PR also adds task documentation forMetricTask
andMetricsControllerTask
.