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-45668: Add --namespace option to ap_verify command line #232

Closed
wants to merge 1 commit into from

Conversation

andy-slac
Copy link
Contributor

The --namespace option can be used together with --db option to run ap_verify against a shared postgres database.

{Summary of changes. Prefix PR title with JIRA issue.}


  • Do unit tests pass (scons and/or stack-os-matrix)?
  • Did you run ap_verify.py on at least one of the standard datasets?
    For changes to metrics, the print_metricvalues script from lsst.verify will be useful.
  • Is the Sphinx documentation up-to-date?

The --namespace option can be used together with --db option to
run ap_verify against a shared postgres database.
Copy link
Member

@kfindeisen kfindeisen left a comment

Choose a reason for hiding this comment

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

Absolutely rejected. ap_verify has a single purpose -- to run standardized datasets, in a self-contained environment, in a reproducible manner. Neither I nor anyone else has the time to maintain the gold-plating of trying to turn it into a general-purpose execution framework; that's what pipetask run is for.

@andy-slac
Copy link
Contributor Author

OK, no problem, it was useful to me to run it in standalone, as I do not know how to run pipetask (and I don't have data on my test machine). Will cancel this PR.

@andy-slac andy-slac closed this Aug 21, 2024
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

Successfully merging this pull request may close these issues.

2 participants