Code review comment for lp:~fluidity-core/fluidity/freesurfacefromnetcdffix

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

Thanks for taking a look Jon - the typo has been fixed. Also noticed a spelling mistake above too. These have all been in there for quite some time now!

I'm working on a test for the functionality - see:
https://code.launchpad.net/~fluidity-core/fluidity/netcdftests

The main test is in place, just looking at how to check invalid files.

I've also discovered a bug in the reader that will go up shortly...

« Back to merge proposal