added inference_only mode (skip target loading, loss computation, and metric logging)#2272
added inference_only mode (skip target loading, loss computation, and metric logging)#2272sbAsma wants to merge 10 commits into
Conversation
…okens_lens for targets; improve empty-batch error handling with max_attempts and skip reason logging
… back to source samples for coordinate/time metadata and override is_spoof for output writing
…r, reshape empty target array to (0, n_channels), and use coord count for targets_lens
|
Beautiful! This is very useful for the pre-operational runs and long-term forecasts |
|
assign me |
evenmn
left a comment
There was a problem hiding this comment.
This looks good to me! I tested running inference with inference_only=True, and is works as expected. We still cannot run inference outside the dataset range, but I suspect this is because the forcings are still taken from the dataset?
I ran inference starting from late december 2023 to january 2024 and it worked for me. |
My bad, I didn't change test_config.end_date, it seems to work now! I guess this works because forcings are not applied continuously for all rollout steps, but that's a separate problem. Nice implementation |
Description
Issue Number
Closes #2256
Is this PR a draft? Mark it as draft.
Checklist before asking for review
./scripts/actions.sh lint./scripts/actions.sh unit-test./scripts/actions.sh integration-testlaunch-slurm.py --time 60