-
-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for Curve Geometry #8
Comments
@zhangjinzhou Can you post your example code that returns the error? Our interface to GDAL may need some updating |
@contra The code is straightforward.
|
@zhangjinzhou Can you include the file then? I need a way to reproduce your error. |
@contra I generated a fgdb with one layer and one geometry. Hope it is enough to explain the issue. |
In fact I would qualify this as a missing feature. Curved geometries are indeed currently not supported by |
I just went through the underlying C++ classes for the various geometry types and I found a huge amount of copying and pasting. Someone should completely rewrite that part, with one base class and some templating/inheritance, I think that the amount of code can be reduced with something like 80% to 90%, all while producing the missing curve geometries in the process. Look in |
The geometries are a major mess. On the JS side, it is a hierarchy of classes that inherit from a base class, on the GDAL C++ side it is a hierarchy too and in |
According to this RFC, gdal is able to handle curve geometry since 2.0. How does the nodejs interface handle this? Currently, the lib returns not supported geometry error.
The text was updated successfully, but these errors were encountered: