Code review comment for lp:~fluidity-core/fluidity/fluidity-initialisation-from_netcdf

Revision history for this message
Adam Candy (asc) wrote :

> I use python code for all of this now, that I'll make available soon.
> Therefore retracting my merge request for now.

How very gentlemanly of you!

I have a branch implementing some of the above, so we could take a look at the two and merge in a combination. The NetCDF input certainly requires an improvement - Jon's also mentioned working on this?

I think my branch with additional debugging for the NetCDf readers (a portion of them) is already in. This does give information on the location of problem reads, warns when values are recast and is accompanied by a suite of tests.

« Back to merge proposal