Skip to content

Add SamudrACE CM4 piControl baseline configs#898

Draft
jpdunc23 wants to merge 5 commits intomainfrom
config/coupled-cm4-piControl
Draft

Add SamudrACE CM4 piControl baseline configs#898
jpdunc23 wants to merge 5 commits intomainfrom
config/coupled-cm4-piControl

Conversation

@jpdunc23
Copy link
Copy Markdown
Member

@jpdunc23 jpdunc23 commented Mar 4, 2026

Adds scripts and configs for baseline training of SamudrACE on the CM4 piControl dataset.

@oliverwm1
Copy link
Copy Markdown
Collaborator

Took a brief look—I think it generally looks good and appreciate the readme. A couple thoughts:

  • how do these scripts related to the existing cm4-ocean, cm4-atmosphere and cm4/coupled-inference-no-target directories in ace/configs/baselines? It is important to avoid duplication and have a single source of truth for configs, so I suspect that maybe all of those directories should be deleted in favor of this new one?
  • why the nested coupled folder? Could just have configs/baselines/cm4-piControl right?

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.

2 participants