Feature #4887
Bug #2244: Pixel (Line/Sample) Projection Offset Issue
let users know when a PDS has needed a pixel referencing override.
This will notify of changes to default offset/mult values in the log file
Description
When a PDS image is loaded into ISIS and needs to be corrected (meaning the default PDF values were updated via $ISIS3DATA/base/translations/pdsProjectionLineSampToXY.def or V2.def), add a line in the label as a comment or at least a line in the processing log which states as much.
maybe something like...
Group = Mapping
...
UpperLeftCornerX = -81276.585842362
UpperLeftCornerY = 222595.37654232
/* default translation for PDS pixel offsets to offsets in meters
/* were modify to correct original for PDS label issues. For overrides
/* see: $ISIS3DATA/base/translations/pdsProjectionLineSampToXY_V2.def
If this is too label invasive, then something like it listed in the log.
pseudo code :
defaults are defined in pds2isis:
xMult = -1.0
yMult = 1.0
xOff = 0.5
yOff = 0.5
check for PDS dataset matches in pdsProjectionLineSampToXY_V2.def to override defaults.
check to see if any defaults changed.
If so, write out warning.
History
#2 Updated by Stuart Sides 10 months ago
- Tracker changed from Recommendation to Feature
- Status changed from Acknowledged to Assigned
- Assignee set to Summer Stapleton
#3 Updated by Summer Stapleton 10 months ago
- Status changed from Assigned to In Progress
#4 Updated by Summer Stapleton 10 months ago
- Status changed from In Progress to Resolved
- Impact updated (diff)
#5 Updated by Trent Hare 10 months ago
I think this a great start but I have a couple recommendations. I tested the new version against MOLA megdr
$ wget http://pds-geosciences.wustl.edu/mgs/mgs-m-mola-5-megdr-l3-v1/mgsl_300x/meg016/megt90n000eb.lbl
$ wget http://pds-geosciences.wustl.edu/mgs/mgs-m-mola-5-megdr-l3-v1/mgsl_300x/meg016/megt90n000eb.img
$ pds2isis from=megt90n000eb.lbl to=out.cub
So I see pds2isis writes to stdout which I thought was a bad idea. I do like that it writes to the print.prt. Based on that fact, I would have assume it would also write to the image's history block (but it is missing). Since we are not writing a comment block in the label, I think it should be associated with the image and written within the image's history block.
BTW, is the "Group = Results" something we have normally used in the past? I don't mind it -- especially if it is used elsewhere.
Recommendations.
(1) remove from stdout
(2) add to image's history block
(3) perhaps split the text into two lines - e.g.
Group = Results # PDS projection offsets and/or multipliers have been changed from the # defaults. The updated values used for importing this image are below. xOffset = -0.5 yOffset = -0.5 xMultiplier = -1 yMultiplier = 1 End_Group
#6 Updated by Trent Hare 10 months ago
- Status changed from Resolved to Closed
I still recommend 1 and 3 from the list above but adding a comment to the cube history (2) sounds tough so we will skip that.
closed.