NOAA-GFDL / pace

Re-write of FV3GFS weather/climate model in Python
Apache License 2.0
12 stars 11 forks source link

[NASA:Update] Distributed dace cache (rework) #16

Closed FlorianDeconinck closed 12 months ago

FlorianDeconinck commented 1 year ago

Purpose

The original work to be able to compile DaCe orchestrated backend strategy was:

The new strategy cleans up code and actually generates the correct caches with any layout. E.g.

The same system should be deploy for gt backends, but is more complex due to the atomic nature of compiling, therefore is not part of this work.

This PR will synchronizes NASA & NOAA forks.

Code changes:

Checklist

Before submitting this PR, please make sure:

FlorianDeconinck commented 1 year ago

We are having a problem with determinism of temporaries. The utest fails sometimes, which sounds bad. But the regression test are passing fine.

FlorianDeconinck commented 1 year ago

We are having a problem with determinism of temporaries. The utest fails sometimes, which sounds bad. But the regression test are passing fine.

This is now solved.

bensonr commented 1 year ago

@FlorianDeconinck - in the orchestration.py source file, the comments highlighted are outdated with the new logic. Please update them.

FlorianDeconinck commented 1 year ago

All issues raised are logged.