Implement consistent model merging across the codebase
@podkidyshev is already working on this.
Since Apr 13, 2026.
- Dominant language
- Python
- Stars
- 99
- Forks
- 62
- Avg merge
- 6d 12h
- Merged PRs (30d)
- 17
Description
Context
In PR #795, we implemented smarter field merging for the nsys configuration by using exclude_unset=True in the model_dump() call. This allows partial overrides of nsys fields in scenario configurations while preserving base values from test definitions.
A similar approach was previously used for reports. However, the current approach for most other fields is "all or nothing" - meaning everything should be set at one level or another.
Proposal
Implement a consistent model merging strategy across the codebase:
- Consider creating a unified
merge_models(tdef, tscenario)function or similar utility - Apply the smarter merge approach consistently for all relevant model fields
- Ensure the merging hierarchy is clear and well-documented
This would provide a friendlier and more intuitive user experience when overriding configuration values.
References
- PR: https://github.com/NVIDIA/cloudai/pull/795
- Discussion: https://github.com/NVIDIA/cloudai/pull/795#discussion_r2788287706
- Requested by: @amaslenn
- Related discussion with: @podkidyshev, @juntaowww
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.