GridGeometry improvements
-------------------------

         Key: GEOT-701
         URL: http://jira.codehaus.org/browse/GEOT-701
     Project: GeoTools
        Type: Improvement
  Components: geometry  
    Versions: 2.2.M1    
 Reporter: Alessio Fabiani


Proposal for some changes to the GridGeometry classes in order to avoid several 
potentially dangerous inconsistencies:
 
1) GridGeometry2D should be responsible of the internal representation of the 
GridCoverage2D grid. In other words the GridGeometry2D constructor would do 
checks for axis swapping and so on, not the GridCoverage2D one. 
 
2) GridGeometry should be more independent from the Coverages. For example we 
could add an Envelope property representing the real world coordinates to the 
GridGeometry class which contains even the GridGeometry CRS. If no CRS is 
specified in the Envelope we assume DefaultWGS84. 
 
3) We could remove all the GeneralGridGeomery constructors except the following:
               
     public GeneralGridGeometry (final GridRange gridRange, final Envelope 
userRange (+ CRS/WGS84 as default), boolean[] reverse, boolean swapXY)
     public GeneralGridGeometry (final GridRange gridRange, final MathTransform 
gridToCoordinate, CoordinateReferenceSystem userCRS)
 
and for the GridGeometry2D:
 
     public GeneralGridGeometry2D (final GridRange gridRange, final Envelope 
userRange (+ CRS/WGS84 as default)) 
     public GeneralGridGeometry2D (final GridRange gridRange, final 
MathTransform gridToCoordinate, CoordinateReferenceSystem userCRS)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to