Use `ProcessBuilder._merge` instead of `ProcessBuilder._update`

aiidateam / aiida-common-workflows

A repository for the implementation of common workflow interfaces across materials-science codes and plugins

https://aiida-common-workflows.readthedocs.io

MIT License

52 stars 30 forks source link

Use `ProcessBuilder._merge` instead of `ProcessBuilder._update` #335

Closed sphuber closed 7 months ago

sphuber commented 7 months ago

Fixes #206

The ProcessBuilder._update method does not properly recurse into nested mapping like ProcessBuilder._merge does. Note that the recursiveness just considers normal Python mappings and will not recurse inside node instances that may behave like mappings, such as Dict nodes.