<div class="gmail_quote">Hello Frank,</div><div class="gmail_quote"><br></div><div class="gmail_quote">2009/6/30 Frank Warmerdam <span dir="ltr"><<a href="mailto:warmerdam@pobox.com">warmerdam@pobox.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div></div><div class="h5">Jorge Arévalo wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
This is the report #5 of GDAL WKT Raster driver: <a href="http://www.gis4free.org/blog/2009/06/27/gsoc-09-weekly-report-5-1906-2606/" target="_blank">http://www.gis4free.org/blog/2009/06/27/gsoc-09-weekly-report-5-1906-2606/</a><br>
<br>
The project can be followed here: <a href="http://trac.osgeo.org/gdal/wiki/WKTRasterDriver" target="_blank">http://trac.osgeo.org/gdal/wiki/WKTRasterDriver</a><br>
</blockquote>
<br></div></div>
Jorge,<br>
<br>
I appologise for not staying on top of my mentoring duties. I am reviewing<br>
the design document at:<br>
<br>
<a href="http://www.gis4free.org/blog/wp-content/uploads/2009/06/gdal-wktrasterdriver-spec-0.2.pdf" target="_blank">http://www.gis4free.org/blog/wp-content/uploads/2009/06/gdal-wktrasterdriver-spec-0.2.pdf</a><br>
<br>
Is this still the most current version of your design? I see Even, Mateusz,<br>
Pierre and others have provided some feedback. Their comments seem good.</blockquote><div><br></div><div>Don't worry. As you said, this is not the current version. Even, Mateusz, Tamas... and myself, have commented a lot of things that must be added. </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
<br>
I would also support the suggestion to put the design in a more malliable<br>
form, such that others can contribute more directly - such as a trac wiki<br>
page. I also think the design needs some further updates from the 0.2<br>
version. I notice you have your own trac now, though it does not seem to<br>
have any public access which I think is unfortunate. Don't hesitate to keep<br>
the design in the GDAL trac.</blockquote><div><br></div><div>OK. I'll use the GDAL trac to create a web version with more information. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
<br>
I would like design to better address the relationship between WKTRaster's<br>
data model and the GDAL data model. So, for instance, you might note things<br>
like:<br>
<br>
1) The fPixelSizeX, fPixelSizeY, fUpperLeftX, fUpperLeftY, fRotationX,<br>
fRotationY values will become the GeoTransform in GDAL, possibly noting<br>
how the transformation will take place. Actually, the f* fields I note<br>
are actually on a single WKTRaster objects after fetching and instead<br>
you really need to populate the Geotransform based on the extent, pixelsize_x<br>
and pixelsize_y fields from the RASTER_COLUMNS table and that<br>
for now only north up images will be supported (unrotated).<br>
<br>
2) Note that WKTRaster uses an SRID lookup into the spatial_ref_sys which<br>
may contain WKT and PROJ.4 representations of a coordinate system, but<br>
that GDAL needs WKT. Note perhaps that the spatial_ref_sys lookup code<br>
from the OGR PG driver can be used essentially unchanged, and that things<br>
only get messy when there is a need to create new rasters as it is sometimes<br>
hard to establish what SRID to use for a given WKT (or whether to create a<br>
new one), but that this matter can also be done using the same approach as<br>
the OGR PG driver.<br>
<br>
--<br>
<br>
In general I was surprised to find no mention of the RASTER_COLUMNS table<br>
and the implications for the WKTRaster driver. To a significant extent<br>
this table was defined to make it easier to write the WKTRaster driver.<br>
It encapsulates a concept of simplistic rasters that can be easily mapped<br>
onto the GDAL data model and that can be efficiently manipulated by GDAL.<br>
<br>
I would like the design talk about steps in implementation. The first<br>
complete step should be a read-only driver that supports<br>
"regular_blocking" type images defined via RASTER_COLUMNS. I would<br>
hope this would be accomplished by the mid-term review.<br>
<br>
After that some additions you could work on are:<br>
<br>
1) Supporting rasters that are not "regular_blocking". I think it will<br>
be hard to make this really efficient, but it is definately doable.<br>
<br>
2) Support in place update for existing rasters.<br>
<br>
3) Support creating new rasters.<br>
<br>
4) Support access to overviews.<br>
<br>
5) Support "outdb" rasters.<br>
<br>
I would like a sense of which of these you hope to do this summer,<br>
in what order, and what you see as the issues with them.<br>
<br>
I would personally be quite happy if you did a bulletproof and efficient<br>
implementation of read-only access to WKTRasters with overviews and<br>
all applicable metadata and access to "outdb" rasters.<br>
<br>
Make sure the project plan includes time to extend the python test suite<br>
with tests for the WKTRaster driver, and to write up user driver<br>
documentation.<br><font color="#888888">
</font></blockquote><div><br></div><div>Understood. Since now, I'm working on an extended version of the document (1.0 this time), describing more in detail all the work, including your comments, and any other information that can be useful. </div>
<div><br></div><div>Many thanks for your comments, I'll keep you (and the list) informed.</div><div><br></div><div>Best regards</div><div>Jorge</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<font color="#888888"><br>
<br>
-- <br>
---------------------------------------+--------------------------------------<br>
I set the clouds in motion - turn up | Frank Warmerdam, <a href="mailto:warmerdam@pobox.com" target="_blank">warmerdam@pobox.com</a><br>
light and sound - activate the windows | <a href="http://pobox.com/~warmerdam" target="_blank">http://pobox.com/~warmerdam</a><br>
and watch the world go round - Rush | Geospatial Programmer for Rent<br>
<br>
</font></blockquote></div><br>