nv-morpheus / Morpheus

Morpheus SDK
Apache License 2.0
309 stars 119 forks source link

[BUG]: vdb_upload config values are ignored #1752

Open dagardner-nv opened 1 week ago

dagardner-nv commented 1 week ago

Version

24.06

Which installation method(s) does this occur on?

Source

Describe the bug.

The code attempts to merge configs originating from CLI flags and from the yaml config, with any CLI values taking precedence.

However there is a problem in that the default values are considered to be a part of the CLI values, thus the config values are never used.

Once resolved, the removal of the documentation for this in PR #1778 should be removed.

Minimum reproducible example

python examples/llm/main.py vdb_upload pipeline --stop_after=1024 --enable_cache --vdb_config_path=examples/llm/vdb_upload/vdb_config.yaml

Relevant log output

Click here to see error details

 [Paste the error here, it will be hidden by default]

Full env printout

Click here to see environment details

 [Paste the results of print_env.sh here, it will be hidden by default]

Other/Misc.

No response

Code of Conduct