Simple tests for UGRID cube printing.#8
Conversation
|
This is just renaming test files? Is that what you meant @pp-mo ? |
Ooops, no. |
pp-mo
force-pushed
the
pp__print_cube_ugrid
branch
from
c9e16ee to
78378c9
Compare
|
Here it is. Sorry for the delay ! |
| ugrid information: | ||
| Mesh0.node - - x | ||
| topology_dimension: 2 | ||
| node_coordinates: latitude longitude |
There was a problem hiding this comment.
So the test is failing on this line. It because these are contained in an unordered set (because there is not order inherent in the file, so to respect that and not to imply any order I just used a set). I could alphabetise them so that the results are consistent ?
There was a problem hiding this comment.
I will fix this in my branch (so I will merge this in the meantime), but I'd appreciate your thoughts on this ^ @pp-mo
There was a problem hiding this comment.
I now sort the list when it is created at load time:
SciTools@d8274ae#diff-3f092d0906b994c134f8f9d64c28652aR235
And so ugrid.node_coordinates now returns a list rather than a set
2 participants
Suggested ultra-simple integration exercise of UGRID cube print strings.