-
Notifications
You must be signed in to change notification settings - Fork 8
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
Question: Scope of the package in terms of raster and vector support #205
Comments
@goergen95 I appreciate your comments and interest in the package. I wasn't planning to add vector support analogous to the interface for raster. The internal functions defined in src/ogr_util.h are the only vector support currently. Those were added for It should be straightforward to support vector using a class-based interface like that for raster (i.e., based around The existing OGR utility functions could be fleshed out and documented pretty easily, if having those would be worthwhile in general (I'm not sure and wasn't planning to specifically). A set of OGR utilities could be added to as needed, but the overall scope would be more limited. Are there certain applications or capabilities that you're interested in for vector data sources? |
Thank you very much for your detailed answer and for taking the time I think it would be best to describe in more detail how my use-case actually looks like. Over at The data requests we receive are really quite diverse and we have to deal Now, the easy answer to such requests would be that we'll have to wait until someone So, to take I am thus less interested in another analytical data model (I think In essence, it seems I hope this is helpful to clarify the context of my question. |
@goergen95 for your use-case I would have a look at the new GTI driver in GDAL (targeted for 3.9) I'll have a closer look but I think there's scope there for your needs. And ... just to throw in I also noticed these, I have nothing more to add atm but just want to make sure more people see the breadth of community interest (that we could potentially consolidate better than has been done in recent years). I'm very keen to get a real API for GDAL in R and extending gdalraster to vector as well is as good as any other means (I use osgeo package in Python via reticulate for my real benchmark, or in Python of course): https://github.com/caiohamamura/gdalBindings-r https://github.com/brownag/ROGRSQL (there's also VicMapR that grew out of bcdata that has a lot in terms of the "lazy tbl" for gdal vector that would be so valuable to have control of, and of course the geoarrow efforts are a good stream to align to for a community foundation). there's so many disparate pieces I actually need to keep notes on them now, so thought I'd throw these in for visibility 🙏 |
@goergen95, thanks for the description and background on Also appreciate the input from @mdsumner on this topic. I wonder about the current read support for vector data sources that I think there are certain advantages to implementing with I'm open to working on a vector interface if there is rationale for it. Not sure exactly what that would look like though. I haven't thought through it enough to really gauge the amount of effort, but a good bit of work obviously. I wonder about time frame for something to still be useful in your project (not that there needs to be specific commitment or anything like that). Other routes could be better but I expect you've explored several options at this point. Another consideration is that the use cases for a low level vector API in R seem somewhat more limited and specialized than for raster. Appreciate any further comments or suggestions. If we do consider implementing something then discussion of a few details would be helpful. |
well, I would really like to (re) implement a lot of vapour in the context of Rcpp modules, and I think a gdalvector counterpart to this package is a good start - I'd benefit a lot from even the bare bones of a vector source info implemented via modules - so if that's easy for you to get started I think it would be easy (easier) for me to pick it up from that start. There's a lot of vapour that is a mess, and it needs to stay backwards compatible so I'm inclined to start again in a new package. I will get to it myself at some point, but fwiw I could probably pick up from progess by others and I'd be happy to stick to a list of features to implement first. I very often go to osgeo.gdal or osgeo.ogr to get what I want, but this package has quickly become very rich in those features 👌 |
@mdsumner, thanks for your feedback. That is good to know. Pending additional comments from Darius, I will follow up on potential proposal for development. Mainly, I'd like to know there is reasonable consensus that adding this functionality would benefit known use cases, and that it makes sense to add it here based on the existing interface. I'm certainly open to hearing anything counter to that. |
Really appreciate the discussion. Agreed, that opinions might differ if "only" giving users pointers instead of a fully-fledged interface is helpful. On the other hand, if this package was to provide modularized low-level access to GDAL/OGR functionality it could actually be used by others creating higher-level interfaces (that is what I understand from your last comment @mdsumner, or did I get something wrong?). I see this already as a great benefit for the community because people could actually just start building things without being doomed to use one of the available data models that might not have been created with their use case in mind. Concerning the functionality, I think anything along the lines of |
Appreciate the helpful discussion. I will work on a draft of proposed bindings to OGR based on the current implementation for raster using Rcpp modules. It will include a description of the interface along with initial attempt at a class definition and method signatures for the core functionality. I'll post that here once ready, for possible review and feedback. Thanks for considering. |
@goergen95 , @mdsumner - |
Thanks, looking forward to dive in. Closing here then. |
Hello @ctoney,
this is awesome work and I am looking forward to watch this package's development more closely.
It really enables a lot of workflows to handle geospatial data in R that were/are lacking in other projects.
This issue is just a question about your plans for future development. First, the name
gdalraster
already suggests its main purpose, I guess. However, I already see first attempts to support OGR?
Personally, I see a great advantage of a R package supporting low-level access to both the raster and vector data models in GDAL/OGR. Is it planned to eventually add full support for both? Or is this already possible and I was just not able to see it?
Appreciate to learn your thoughts on this.
The text was updated successfully, but these errors were encountered: