Tiles are empty on the Western U.S.

ngageoint / elasticgeo

ElasticGeo provides a GeoTools data store that allows geospatial features from an Elasticsearch index to be published via OGC services using GeoServer.

GNU General Public License v3.0

168 stars 85 forks source link

Tiles are empty on the Western U.S. #80

Open shree665 opened 6 years ago

shree665 commented 6 years ago

We have run into the issue that data is missing over the western U.S. We saw same issue when we open a layer using openLayer in GeoServer as well as in Leaflet. But the issue won't happen missing data over western U.S issue when we use postGis as a backend. So, I did the following changes to the code.

updated clipToWorld() method which seems to have a bug and returning empty tiles for West coast of USA
added elasticsearch connection timeout on rest client

I can provide script and leaflet html code to test the issue if needed.

shree665 commented 6 years ago

I have updated the failed tests. I think those two failed tests were wrong before. testBBOXCoveringDateline() test method should be asserted to one point because wifiAccessPoint.json only has one record which is inside the provided bounding box. On the other hand, testBBOXBeyondDateline() test method should be asserted to zero point. am I missing anything on those tests? It don't want to sound crazy here.

Please let me know if I need to do anything else.

sjudeng commented 6 years ago

Thanks for adding that test. testBBOXCoveringDateline is testing that both 45,179 and 45,-179 are included when the bbox crosses the dateline (e.g. 178, -98, 182, 98). Likewise testBBOXBeyondDateline is testing that the 45,-179 point is included when the bbox is on the other side of the dateline with coords in [0,360] instead of [-180,180] (e.g. 180.5, -98, 182, 98).