nicmichael / sgt Goto Github PK

I feel the color picked in fillSquare method is not correct.

The reasons is it seems it is assuming that number of contour levels and number of colors in color map are equal. which may not be the case.

Better solution will be to pick color by using following:

//col = cmap.getColorByIndex(cindex + 1);
if (zmin > zlevp1 || zmax < zlev) {
continue;
}
try {
col = attr_.getColorMap().getColor(clevels.getLevel(cindex + 1));
} catch (ContourLevelNotFoundException ex) {
col = cmap.getColorByIndex(cindex + 1);
}
Kindly correct me if I am incorrect. I have tested it never goes into catch. Just a fallback option.
I am attaching the current and proposed areafill snapshots by running JGridDemo.
The proposed matches closely to the raster in terms of color map.