Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#4165 closed defect (fixed)

gctp wrapper in the hdf4 driver does not apear to work

Reported by: winkey Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: unspecified
Severity: normal Keywords: gctp
Cc: dron

Description

gctp wrapper does not work properly.

although frank and even does not seem to think this code is even used in the driver anywhere it is very useful code for replacing gctp in applications like swath2grid

Brian

Attachments (1)

gctp_wrap.diff (6.8 KB ) - added by winkey 13 years ago.
patch to fix the gctp wrapper

Download all attachments as: .zip

Change History (6)

by winkey, 13 years ago

Attachment: gctp_wrap.diff added

patch to fix the gctp wrapper

comment:1 by winkey, 13 years ago

gctp expects to receive and returns lat/lon in radians.

the old version leaked large amounts of ram in the inv transformation.

the old version was very costly to use, because the transform was created for every point transformation.

comment:2 by warmerdam, 13 years ago

Cc: dron added
Keywords: gctp added

Brian,

I would suggest just committing it in trunk.

I am cc:ing Andrey who may have interest.

comment:3 by dron, 13 years ago

It looks fine to me, please go on commiting it.

Just for record: the wrapper itself is not used in the HDF4 driver, but HDF-EOS library uses it (instead of GCTP). We are not using HDF-EOS functions which depend on coordinate conversions though.

Best regards,
Andrey

comment:4 by winkey, 13 years ago

Resolution: fixed
Status: newclosed

commited to trunk

comment:5 by warmerdam, 13 years ago

The change was r22751.

Brian, please try to include the ticket number (prefixed with #) in the commit message and the revision number (prefixed with 'r') in the ticket so we can track back and forth.

Thanks

Note: See TracTickets for help on using tickets.