Justin Deoliveira wrote: > So basically you are talking about making the context a Hints instance. > I am not sure I like this approach as it tends to spiral out of control > with no constraint as you can add anything to the hints and it may or > may not be regarded. > Fair enough - valid point all. Just would of been a quick fix and less ideas to introduce. > I can think of two additional approaches for the crs problem which are > more explicit. > > 1. Create a "ReprojectionExpression" (for lack of a better name) which > will take another expression and ensure it is evaluated and reprojected > accordingly. > Actually I think we should already have this in our mix - as a Reprojection Function; this would be: a) a good clean integration at a user visible level b) really useful when putting together SQL for things like Oracle or Postgis....
Downside is forcing users to know about projections ... it would be magic if it just worked form them. > 2. Make spatial expressions know about crs, and have part of their > contract be that things should be reprojected into the set CRS. > That is what I would prefer; and for the Oracle and GeoAPI Geometry they can do just that - but for the JTS Geometry and Postgis Geometry we are out of luck ... For property expressions we can get the following to work with a bit of luck. Object geom = EXPR.evaulate( object ); // geom? CRS crs = EXPR.evaulate( object, CoordinateReferneceSystem.class ); But for literals we are always going to be troubled unless they specify .... so close but yet so far. Well we can always place the CRS in the JTS Geometry user object and hope for the best. Jody > > > >> Jody >> >>> I would ask Justin to comment (please) on the second point; and will >>> note the existing fix for Expression.evaulate( Object, class ) allows >>> the spatial functions to "pull" down JTS Geomertry as the system has >>> been taught how to convert - for the wrapping based solution this is >>> an easy win (make a converter to unpack and then everything will >>> work). This may let you put off point #2. >>> >>> Cheers, >>> Jody >>> >>> >>> >>>> I commited Sanjay Jena's work on ISO 19107 implementation: >>>> >>>> http://svn.geotools.org/geotools/trunk/gt/modules/unsupported/geometry/ >>>> >>>> This a work in progress, and is not yet part of the Maven build. I >>>> will work on the Maven configuration later. >>>> >>>> Note to Sanjay: I renamed the following packages: >>>> >>>> org.geotools.geometry.featgeom >>>> geometry.featgeom >>>> >>>> into >>>> >>>> org.geotools.geometry.iso >>>> >>>> Since "featgeom" is somewhat redundant with "geometry", and "iso" is >>>> complementary with "jts" (an other org.geotools.geometry subpackage). >>>> However we may drop the "iso" name completly in a future version and >>>> put the classes straight at the "org.geotools.geometry" level; I'm >>>> not yet sure and for sure it will depends if you agree or not :). >>>> >>>> I also moved the "lineSegmentsData.dat" and "lineStringData.dat" >>>> files into the >>>> "src/test/resources/org/geotools/geometry/iso/test-data" directory. >>>> However I don't know which code to change for that, since I didn't >>>> any found "lineFooData.dat" occurence in the test directory. >>>> >>>> Apart for the package name changes, I didn't changed anything to the >>>> code. >>>> >>>> Thanks a lot for sharing your work!! >>>> >>>> Martin >>>> >>>> >>>> ------------------------------------------------------------------------- >>>> >>>> Take Surveys. Earn Cash. Influence the Future of IT >>>> Join SourceForge.net's Techsay panel and you'll get the chance to >>>> share your >>>> opinions on IT & business topics through brief surveys - and earn cash >>>> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV >>>> >>>> _______________________________________________ >>>> Geotools-devel mailing list >>>> [email protected] >>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel >>>> >>>> >>> ------------------------------------------------------------------------- >>> Take Surveys. Earn Cash. Influence the Future of IT >>> Join SourceForge.net's Techsay panel and you'll get the chance to >>> share your >>> opinions on IT & business topics through brief surveys - and earn cash >>> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV >>> _______________________________________________ >>> Geotools-devel mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/geotools-devel >>> >>> >> !DSPAM:1004,45663fc1159641336712104! >> >> > > > ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
