search

coronis-computing / heightmap_interpolation

GNU Lesser General Public License v3.0
3 stars 2 forks source link

Issue with some kml #3

Open cyrilleponcelet opened 2 years ago

cyrilleponcelet commented 2 years ago

Hello, We are working to integrate your algorithm into the Globe ( https://doi.org/10.17882/70460 ) software and we're having some issues with the kml format taken into account. Some kml file are not working like the ones produces by Globe. In such case a crash occurs in apps/netcdf_data_io.py:114

This happens when kml contains polygons instead of line string. Please find below the content of a kml file having such an issue (you probably need to see the raw markdown to copy paste the content)

<?xml version="1.0" encoding="UTF-8"?>

SSV_Zone Zone 0 1 normal 0 clampToSeaFloor 45.36623707872974,-12.806073996982407 45.36376406248019,-12.809128533617994 45.3659812337404,-12.809086710022665 45.36470206827514,-12.81113703032293 45.36649287411026,-12.81122072715687 45.36559745558184,-12.812559722617713 45.369221722136096,-12.8117228493935 45.36657815732431,-12.810049109726915 45.36738828097198,-12.807998789678642
ricardcd commented 2 years ago

Hi, The issue should be fixed now in master. I was expecting the areas in the KMLs to be in the format output by QGis (as a single MultyPolygon geometry). Now it should accept Polygons. Please let me know if this change solves the issue.

devos0024 commented 2 years ago

Hello, We have integrated the last version in Master. I'm glad to confirm that the changes made have solved the issue. Our KMLs are now correctly processed