Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ap_pipe
-
Story Points:6
-
Epic Link:
-
Sprint:AP F20-5 (October), AP F20-6 (November)
-
Team:Alert Production
-
Urgent?:No
Description
make_apdb.py is a non-Task command-line script. To make it easy to keep database configuration consistent between it and ap_pipe.py, make_apdb.py takes exactly the same config arguments as ap_pipe.py. In Gen 2, it does this by internally reusing ApPipeConfig and two custom argparse.Action classes from CmdLineTask. All of these classes will be removed along with Gen 2.
Develop a new make_apdb.py that matches the interface used to configure the AP Pipeline (or at least the APDB) with the pipetask activator:
pipetask run -c label:config_var.sub_config=6
|
pipetask run --config label:config_var.sub_config=6
|
pipetask run -C label:path
|
pipetask run --config-file label:path
|
pipetask run --pipeline path # May configure any component, including APDB
|
It may be useful to wait until DM-21939 is done, so that we can see exactly what the new ap_pipe.py interface will look like. The current make_apdb.py will work with the Gen 3 pipeline for as long as CmdLineTask is also supported.
Unfortunately, the Gen 3 equivalents of ConfigFileAction and ConfigValueAction do not do any config processing, which is instead tightly coupled to the activator code itself. So the Gen 3 version will have to find a different way to give users a consistent UI.
Attachments
Issue Links
- is blocked by
-
DM-22741 Remove ap_association subtasks from ap_pipe. Change to single DiaPipelineTask subtask.
- Done
- relates to
-
DM-27461 Unclear error messages in ConfigOverrides
- To Do
-
DM-27462 Add --config-file option alias to CmdLineTask
- Done
-
DM-21939 Create Gen 3 AP Pipeline
- Done
-
DM-24436 Use obs and camera overrides in make_apdb.py
- Won't Fix
- mentioned in
-
Page Loading...
The new make_apdb.py script should support obs package overrides for the APDB, possibly through apPipe.py or diaPipe.py, as originally proposed on
DM-24436.