Skip to content

Add multi-fidelity/modality support#313

Draft
radka-j wants to merge 28 commits into
mainfrom
308-multi
Draft

Add multi-fidelity/modality support#313
radka-j wants to merge 28 commits into
mainfrom
308-multi

Conversation

@radka-j

@radka-j radka-j commented Apr 7, 2026

Copy link
Copy Markdown
Member

Closes #308

We have 2 scripts in notebooks/ for the full workflow:

  • generate_reaction_diffusion.py
  • train_reaction_diffusion.py

QUESTIONS and TODOs:

  • at the moment we do end-to-end training on the encoder-processor-decoder but we might want to add an option to train the encoders independently, then freeze them and only train the rest of the model (attention + processor + decoder)

sgreenbury and others added 5 commits March 28, 2026 07:15
- Break run_evaluation into focused phase functions (_run_test_metrics,
  _run_rollout_videos, _run_rollout_metrics, _run_rollout_per_timestep_metrics,
  _write_final_csvs) to reduce complexity and improve readability
- Extract _resolve_metric_fns to eliminate duplicated metric resolution logic
  between test and rollout metric setup
- Unwrap Fabric-wrapped model once and pass unwrapped model to functions that
  access model internals (parameter counting, benchmarks) instead of relying
  on Fabric proxy attribute access with pyright ignores
- Add depth limit to _unwrap_module to guard against circular wrapper chains
- Extract _setup_fabric, _load_model_checkpoint, and _resolve_eval_paths
  helpers from run_evaluation
Co-authored-by: Radka Jersakova <r.jersakova@gmail.com>
Co-authored-by: Paolo Conti <ContiPaolo@users.noreply.github.com>
Co-authored-by: Paolo Conti <ContiPaolo@users.noreply.github.com>
Co-authored-by: Radka Jersakova <r.jersakova@gmail.com>
Co-authored-by: Paolo Conti <ContiPaolo@users.noreply.github.com>
Co-authored-by: Radka Jersakova <r.jersakova@gmail.com>
@radka-j radka-j marked this pull request as ready for review April 7, 2026 09:05
@radka-j radka-j marked this pull request as draft April 7, 2026 09:05
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.

Implement support for multi-fidelity and multi-modality

3 participants