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-22663: Reimplement make_apdb.py for Gen 3 #62
Conversation
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 this solves the issue at hand, but I do have a few lingering concerns about how we ensure users wind up with self-consistent configs when making an APDB vs. running the AP pipeline. Thank you for updating the docs and adding tests!
c8f6f0a
to
903328e
Compare
903328e
to
cc33cec
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.
Thanks for simplifying the config override situation!
The previous parser used the CmdLineTask code, but the latter will be going away when Gen 2 is removed.
The switch from ApPipeConfig to ApdbConfig is a breaking change, but one that was agreed to by the users.
This is likely to be a very common user error during the transition to Gen 3.
cc33cec
to
c43f447
Compare
This PR removes the
make_apdb.py
script's dependence onpipe_base
and onApPipeConfig
, making it forward-compatible with Gen 3. As a side effect, the script now loads configs at a lower level, requiring changes to calling scripts.