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-43414: Update make_apdb module for changes in APDB API #173
Conversation
`make_apdb` will be replaced by `apdb-cli` sub-command, for now we keep it for compatibility reasons, but update it for new APDB API. This script depends on using pex_config for Apdb configuration, that will be replaced eventually with something different, we may want to drop this script before that change.
15baeec
to
9b11acb
Compare
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 recommend emitting a deprecation warning from make_apdb.py
, otherwise people will keep using it indefinitely.
On that note, please update the docs with the recommended idioms:
- doc/lsst.ap/pipe/pipeline-tutorial.rst
- doc/lsst.ap/pipe/apdb.rst
- The comments at the top of pipelines/*/ApPipe*.yaml
- https://github.com/lsst-dm/prompt_processing/blob/main/doc/playbook.rst#databases
a3faf0e
to
bf53ea9
Compare
@kfindeisen, I added deprecation, updated docs and pipelines. PR for prompt_processing playbook is here: lsst-dm/prompt_processing#151 |
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 think the main user guide for apdb-cli
should be in the same package (i.e., dax_apdb
), but happy to leave that for later. DM-43416 will require another rewrite of this text anyway. 🙂
bf53ea9
to
6187171
Compare
make_apdb
will be replaced byapdb-cli
sub-command, for now we keep it for compatibility reasons, but update it for new APDB API. This script depends on using pex_config for Apdb configuration, that will be replaced eventually with something different, we may want to drop this script before that change.