Opened 14 years ago
Closed 14 years ago
#3355 closed defect (fixed)
Fixes to variations on PDS Image pointer and implementation of LINE_PREFIX_BYTES
Reported by: | gaige | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | 1.7.2 |
Component: | GDAL_Raster | Version: | 1.7.0 |
Severity: | normal | Keywords: | PDS |
Cc: |
Description
This patch fixes a couple of bugs in the current PDS implementation, namely:
- LINE_SAMPLES is used as the byte width for a row, when RECORD_BYTES is actually the byte width of a row, the LINE_SAMPLES merely describes how many data points there are on the row
- Add support for LINE_PREFIX_BYTES, which is resident in certain test files that I have found from NASA
- Modified support for PDS Image Pointer variations (#3177) to comply with the documentation
- Fixed a bug that caused the pointer variation fix in 3177 not to work due to the quotation marks being removed from the line
I've enclosed the diff files against 1.6.3 and a couple of different test sets. Please feel free to contact if there are any questions.
Attachments (3)
Change History (8)
by , 14 years ago
Attachment: | pdsdataset.cpp-diffs added |
---|
comment:1 by , 14 years ago
Keywords: | PDS added |
---|---|
Milestone: | 1.6.4 → 1.7.1 |
Resolution: | → fixed |
Status: | new → closed |
comment:2 by , 14 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I've upgraded to 1.7 here on my test bed and a number of the changes fell out when it was pulled into 1.7 and trunk.
I'm reopening this, and I will provide diffs based on 1.7 and trunk in the near future. In particular, the <BYTES> support fell out, as did support for images which have quotation marks in their pointer specifiers.
I can provide test files for these if you like.
comment:3 by , 14 years ago
Version: | 1.6.3 → 1.7.0 |
---|
Here are the diffs for 1.7. They address the following problems:
- N/A units, which I've found in some mars PDS files
- The 3 image pointer formats IMAGE = ("blah.img") , IMAGE= ("blah.img",1) and IMAGE = ("blah.img", 1 <BYTES> )
- The previous problem with the use of the IMAGE decoder removing the quotes around the file name, resulting in the file name not being recognized (due to the test for osQube[0]=='"'
-Gaige
comment:4 by , 14 years ago
Unfortunately; this patch does not apply for me in trunk or 1.7 branch:
warmerda@gdal64[53]% patch < ~/Desktop/pds-diffs can't find file to patch at input line 2 Perhaps you should have used the -p or --strip option? The text leading up to this was: -------------------------- |==== //import/GDAL/frmts/pds/pdsdataset.cpp#5 (text) - //depot/GDAL/frmts/pds/p dsdataset.cpp#8 (text) ==== content -------------------------- File to patch: pdsdataset.cpp patching file pdsdataset.cpp Hunk #3 succeeded at 604 (offset 3 lines). Hunk #4 FAILED at 610. Hunk #5 succeeded at 632 (offset 3 lines). Hunk #6 succeeded at 698 (offset 11 lines). Hunk #7 FAILED at 715. Hunk #8 FAILED at 815. Hunk #9 FAILED at 821. Hunk #10 FAILED at 827. Hunk #11 succeeded at 848 (offset 16 lines). 5 out of 11 hunks FAILED -- saving rejects to file pdsdataset.cpp.rej
I've skimmed the patch, but it lacks sufficient context to properly hand apply. I don't really see how the current code could not be working for BYTES. I haven't review the other aspects closely.
comment:5 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
My apologies for the confusion. This was reopened in error due to 1.7.0 vs 1.7.1 introduction of these changes.
I've closed it as 1.7.1 appears just fine, thank you.
-Gaige
Patches against 1.6.3