search

CRAVA / crava

6 stars 13 forks source link

Added error message for reading surface #257

Closed hgolsen closed 7 years ago

hgolsen commented 7 years ago

Error when the xyz-surfaces does not have even sampling.

alfbr commented 7 years ago

The error message is very vague. I believe CRAVA actually supports one missing point in a surface, but not two neighbouring points missing. It seems you will throw regardless with this patch?

hgolsen commented 7 years ago

This isn't missing values, but uneven number of points/irregular grid. This is the part of the code where we read the surfaces before we check if some of the values are missing.

It looks like the surface is something like this, x x x x x x x x x x
x x xx xx x x xx xx x x

And this causes problems for the routine that reads the surfaces as we try to fit it into a regular grid/surface.