Skip to content

draft of changes to make inference with develop-ssl trained srdrwfy6 … - #2675

Draft
shmh40 wants to merge 4 commits into
develop-ssl-diffusion-v1from
shmh40/dev/fix-srd-inference-develop-ssl-diffusion-v1
Draft

draft of changes to make inference with develop-ssl trained srdrwfy6 …#2675
shmh40 wants to merge 4 commits into
develop-ssl-diffusion-v1from
shmh40/dev/fix-srd-inference-develop-ssl-diffusion-v1

Conversation

@shmh40

@shmh40 shmh40 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

…work

draft PR to see changes necessary to make:

uv run inference --from-run-id srdrwfy6 --options test_config.samples_per_mini_epoch=16 test_config.output.num_samples=16 test_config.start_date=2023-10-01T00:00 test_config.end_date=2023-12-31T00:00 test_config.time_window_step=06:00:00

work on develop-ssl-diffusion-v1 branch

Issue Number

Is this PR a draft? Mark it as draft.

Checklist before asking for review

  • I have performed a self-review of my code
  • My changes comply with basic sanity checks:
    • I have fixed formatting issues with ./scripts/actions.sh lint
    • I have run unit tests with ./scripts/actions.sh unit-test
    • I have documented my code and I have updated the docstrings.
    • I have added unit tests, if relevant
  • I have tried my changes with data and code:
    • I have run the integration tests with ./scripts/actions.sh integration-test
    • (bigger changes) I have run a full training and I have written in the comment the run_id(s): launch-slurm.py --time 60
    • (bigger changes and experiments) I have shared a hegdedoc in the github issue with all the configurations and runs for this experiments
  • I have informed and aligned with people impacted by my change:
    • for config changes: the MatterMost channels and/or a design doc
    • for changes of dependencies: the MatterMost software development channel

@clessig

clessig commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

@shmh40 : could you open it against #2674 and discuss with @Jubeku . There's some overlap but some things are also different.

@clessig

clessig commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

And @shmh40 : this PR is based on develop-ssl and not your branch where sbr... was trained on

Jubeku added 2 commits July 23, 2026 11:00
Restore target expansion logic to handle multiple forecast steps and improve clarity in the warning messages.
…ev/fix-srd-inference-develop-ssl-diffusion-v1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants