COSIMA / esmgrids

Python representation of Earth System Model grids.
Apache License 2.0
4 stars 7 forks source link

Add Github CI #8

Closed anton-seaice closed 7 months ago

anton-seaice commented 7 months ago

@micaeljtoliveira

I tried to move the existing tests over the github actions, and found many failures.

Most of them seem to be about imports, but some are undefined names that are not clear at all where they have come from!

e.g.

./build/lib/esmgrids/fv300_grid.py:26:32: F821 undefined name 'sys'
                          file=sys.stderr)
                               ^
./build/lib/esmgrids/oras_grid.py:36:47: F821 undefined name 'zdescription'
                                       levels=zdescription)
                                              ^
./build/lib/esmgrids/um_grid.py:20:34: F821 undefined name 'os'
        self.um_restart_output = os.path.join(output_dir,
                                 ^
./build/lib/esmgrids/um_grid.py:21:47: F821 undefined name 'os'
                                              os.path.basename(um_restart))
                                              ^
./build/lib/esmgrids/um_grid.py:22:34: F821 undefined name 'os'
        self.lfrac_filename_nc = os.path.join(output_dir, 'lfrac.nc')
                                 ^
...
anton-seaice commented 7 months ago

Oops you already made it this far :) https://github.com/COSIMA/esmgrids/pull/7