You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
fix(training)!: Refactor configuration by introducing system schema with hardware, paths, and files subschemas (#598)
# Description
This PR reorganizes the configuration structure by introducing a new
top-level schema called system, which groups the subschemas hardware,
storage, and files (see [issue
#513](#513)).
Previously, paths and files were defined under hardware, which was
confusing since they describe filesystem layout and not than compute
resources.
It also wasn’t clear whether paths refer to directories or files. It has
now been renamed to storage, clarifying its role as the definition of
directory structure for inputs, outputs, and logs.
```
system/
├── hardware.yaml
├── input.yaml
└── output.yaml
```
The PR also isolates the concatenation logic for paths in the pydantic
scheme so we don't need to write out the full paths for all
outputs/logs/etc in each field. This is very brittle and used to happen
in both code throughout the framework and inside the nested
configuration files. This is now isolated to happen in only one place.
<!-- readthedocs-preview anemoi-training start -->
----
📚 Documentation preview 📚:
https://anemoi-training--598.org.readthedocs.build/en/598/
<!-- readthedocs-preview anemoi-training end -->
<!-- readthedocs-preview anemoi-graphs start -->
----
📚 Documentation preview 📚:
https://anemoi-graphs--598.org.readthedocs.build/en/598/
<!-- readthedocs-preview anemoi-graphs end -->
<!-- readthedocs-preview anemoi-models start -->
----
📚 Documentation preview 📚:
https://anemoi-models--598.org.readthedocs.build/en/598/
<!-- readthedocs-preview anemoi-models end -->
---------
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: anaprietonem <[email protected]>
Co-authored-by: Ana Prieto Nemesio <[email protected]>
Co-authored-by: Dieter Van den Bleeken <[email protected]>
Co-authored-by: Mario Santa Cruz <[email protected]>
0 commit comments