-
Notifications
You must be signed in to change notification settings - Fork 73
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
How to incorporate proj 5.0 changes? #12
Comments
In general, yes, it does seem like proj4J will have to track PROJ.4 at some point. This seems like an ideal thing for LocationTech to find funding for. |
Does incorporating the pipeline pattern require adding 4D support? |
It does appear that pipeline is implemented separately for 2D, 3D, and 4D. |
Being a fan of simplicity, my first inclination would be to simple add a t ordinate to the existing |
Now that proj4 has stopped using source data files compatible with the existing |
(This might be better opened as a separate issue BTW)
I have a strong personal dislike to the use of SQLite, since it is (a) complex and heavyweight and (b) native code. So -1 on adopting it from me. That said, I realize that this will make tracking PROJ harder. I do wonder why SQLite has been deemed necessary for PROJ. Is it because that's what EPSG uses? To avoid the chore of writing some query/join code? |
What should the process be for starting to think about incorporating proj’s 5.0 overall API changes (in addition to the testing changes being addressed by #9)?
It seems like proj4j is going to eventually need to incorporate proj’s new pipeline and some port of the changed API’s functionality to be able to keep up with upstream’s changes, especially as projections start to be redefined with this new process. Is a good starting point to add time in as a fourth variable to projected coordinates and start implementing the various pipeline operations accordingly? Would it be better to make a new object like ProjCoordinate4D for this (replicating proj’s model of new things living in the “PJ_” api), or would it be better to add “t” into the existing ProjCoordinate and feather in that functionality as is?
The text was updated successfully, but these errors were encountered: