Jody Garnett wrote:
> Saul Farber wrote:
>> Jody Garnett wrote:
>>> Saul this API (ImageReaders) kind of is what you want to make it to be;
>> Or do you mean that the API as defined in modules/coverage (I think 
>> you referred to it as GCE below) is open-ended enough that making a 
>> new reader/format is kind-of amorphous?
> I was meaning that the GeoTools code here does not really have champion; 
> so if you want to lock things down to the point you can reuse your 
> readers then I am pretty sure everyone will be open to the idea.

Cool this all makes more sense then.  So far the geotools 
interfaces/APIs make total sense to me and the only changing that needs 
to be done is up in Geoserver land.


> Right now people often skip the GridCoverageExchange class and go right 
> to the factory (or even implementations). I think our source code 
> examples show this technique in use.

Yeah, I'm just basically copying the ArcGrid reader.  it's very cleanly 
documented and used.


> I was thinking of GridCoverageReader - this method takes an object 
> defining the source:
> - 
> http://javadoc.geotools.fr/2.3/org/geotools/data/coverage/grid/file/FileSystemGridCoverageExchange.html#getReader(java.lang.Object)
> 
> That object can be an array if you have a bunch of stuff you set all the 
> time....

Exactly.  This is exactly what I'm doing now...the only change is that 
geoserver needs to allow storing the source as something other than a 
String.  But this is exactly what I do right now.

> 
> You can then reserve the read method for the things that only very on 
> each call:
> - 
> http://javadoc.geotools.fr/2.3/org/geotools/gce/geotiff/GeoTiffReader.html#read(org.opengis.parameter.GeneralParameterValue[])
> 

And this is what geoserver does, exactly.  Calls read with the things 
that change (requested image size, requested image extent) each time.

> I am going to have to bow out here as I am too far away from the details 
> to be useful.
> Jody

You're more than useful, you've give me great confidence that I'm on the 
right path!  Sorry about the confusion between "Reader" and "Reader". 
It's a killer!

--saul


-------------------------------------------------------------------------
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

Reply via email to