<div>Thanks Frank, I will define product-specific methods just to fetch the metadata in the open method.</div>
<div>I will use the hdf4imagedataset code as a guide on how to process the metadata.(I didn't realized earlier that hdf4 implements something similar to what I asked :P)</div>
<div> </div>
<div>I still need to solve a few things before posting the patch, I look foward incorporate the patch into the project.</div>
<div> </div>
<div>Regards<br><br></div>
<div class="gmail_quote">2011/7/4 Frank Warmerdam <span dir="ltr"><<a href="mailto:warmerdam@pobox.com">warmerdam@pobox.com</a>></span><br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">On Mon, Jul 4, 2011 at 9:16 AM, Alex Mantaut<br>
<div class="im"><<a href="mailto:alexmantaut@suremptec.com.ar">alexmantaut@suremptec.com.ar</a>> wrote:<br></div>
<div class="im">> Hi Frank,<br>> In that case all the handling for the CSK product should go on the<br>> Open() method from hdf5imagedataset? Is there any way to avoid polluting the<br>> hdf5imagedataset with all the subproduct's constants?<br>
<br></div>Generally I would just use the constants inline rather than predefine<br>them for this purpose. If you find the product stuff is getting really<br>bulky you could move it into a separate source file, but I'd suggest<br>
avoiding extra specific classes.<br>
<div class="im"><br>> Shall I define Product<br>> specific private methods as: getCSKGeotransform() to get the information?<br>> regards<br><br></div>Normally in the open we would do a product-specific method to fetch<br>
metadata and it would set the metadata data members, geotransform and<br>other similar stuff rather than having many product specific methods for<br>particular data items.<br><br>I can try to review your patch when ready.<br>
<div>
<div></div>
<div class="h5"><br>Best regards,<br>--<br>---------------------------------------+--------------------------------------<br>I set the clouds in motion - turn up | Frank Warmerdam, <a href="mailto:warmerdam@pobox.com">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></div></div></blockquote>
</div><br><br clear="all"><br>-- <br>--<br> Alex Mantaut<br>SUR Emprendimientos Tecnológicos<br><br>Perú 345 Piso 5to Oficina "B" (C1067AAG)<br>Ciudad de Buenos Aires, Argentina<br>Tel. +54 (11) 4342-2976/84<br>
<a href="mailto:alexmantaut@suremptec.com.ar" target="_blank">alexmantaut@suremptec.com.ar</a><br><a href="http://www.suremptec.com/" target="_blank">www.suremptec.com</a><br>