Skip to content

Update YAML editing window#55

Merged
erwanp merged 2 commits into
mainfrom
fix/open-unfolded-yaml
May 3, 2026
Merged

Update YAML editing window#55
erwanp merged 2 commits into
mainfrom
fix/open-unfolded-yaml

Conversation

@erwanp
Copy link
Copy Markdown
Contributor

@erwanp erwanp commented May 3, 2026

  • Fixes composite unfold collision (child reactors created from parent nodes) when saving from the YAML modal or re-parsing after sync: validate_config drops __synthesized, so the merge path now derives synthesized IDs by re-normalizing the original YAML and diffing against authored list IDs.
  • Merged output no longer injects top-level keys that were absent on disk (e.g. empty settings / output from convert_to_stone_format).
  • Property merge skips normalization-only defaults (e.g. propagated pressure) while still adding GUI-only properties that differ from a fresh normalize of the original file.
  • Network list order follows the original YAML; GUI-added nodes/connections are appended at the end.
  • Dump uses sequence indentation inferred from the source file so list layout matches the on-disk style.

Note that Units can be freely modified from within the YAML text file (GUI editor window) and it reflects/converts immediatly .

@erwanp
Copy link
Copy Markdown
Contributor Author

erwanp commented May 3, 2026

(merging despite make qa failing)

@erwanp erwanp merged commit 6ec0938 into main May 3, 2026
6 of 7 checks passed
@erwanp erwanp deleted the fix/open-unfolded-yaml branch May 3, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant