-
Notifications
You must be signed in to change notification settings - Fork 2
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
Can I propose some missing casts ? #5
Comments
Hi @Remi-C .Thanks for your interest. Yes, we're certainly open to a PR or some other form of collaboration. Only issue I'm not sure about at the moment is introducing a dependency on PostGIS for this extension (although all our databases runs Postgis already, so it might make sense). Currently thinking if the PostGIS functions should be a separate extension. Or I might be able to rewrite some functions to get rid of the PosGIS functions. I believe a lot of your functions are about reconverting from SFTI to something else. While we mostly had functions to go from something to SFTI? |
@Remi-C I've implemented casts from sfti to int and float in the sfti extension itself.
Basically, this determines the middle of the sfti core and returns that as a float or rounds it as an int. Is this sufficient for your use case or did you want something else? I notice you solved this by calculating the centroid of the geom. This might give different results? |
Hey again @koenedaele,
I settled on this version of simple fuzzy intervals.
For the moment it works very well (except installation which is a little bit broken, no big deal)
However I ended up needing few functions more,
such as basic casts (sfti to record, to array, to float, to numrange ),
as well a a cast to geometry (an easy way to use indexing).
Here is the list of the new functions
Would you be open to a PR ?
Cheers,
Rémi-C
The text was updated successfully, but these errors were encountered: