Project

General

Profile

Bug #4530

Bug #2244: Pixel (Line/Sample) Projection Offset Issue

Pixel Offset - isis2pds

Added by Jason Laura about 1 year ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
John Bonn
Category:
Applications
Target version:
Impact:
Software Version:
Test Reviewer:

Description

DoD for the export version:
* Change 2 lines of code and run the tests
* Identify what broke
* Research the 'break' for each to see if you have actually fixed it - this is documentation
* Clean up user test tickets if it did not break anything

History

#1 Updated by Jason Laura about 1 year ago

  • Status changed from New to Acknowledged

#2 Updated by Trent Hare about 1 year ago

  • Category set to Applications

As pds2isis is updated then exporting will need to also be updated.
http://isis.astrogeology.usgs.gov/Object/Programmer/_process_export_pds_8cpp_source.html

from:
00776 lineOffset += 0.5; // Add half a line to get to the center of (1,1)
00781 sampleOffset += 0.5; // Add half a sample to get to the center of (1,1)

to

00776 lineOffset -= 0.5; // Subtract half a line to get to the center of (1,1)
00781 sampleOffset -= 0.5; // Subtract half a sample to get to the center of (1,1)

A great test image is LOLA DEM since it is so small, simple and global 1440x720
PDS image: http://pds-geosciences.wustl.edu/lro/lro-l-lola-3-rdr-v1/lrolol_1xxx/data/lola_gdr/cylindrical/img/ldem_4.img
detached label: http://pds-geosciences.wustl.edu/lro/lro-l-lola-3-rdr-v1/lrolol_1xxx/data/lola_gdr/cylindrical/img/ldem_4.lbl
parent link: http://pds-geosciences.wustl.edu/missions/lro/lola.htm

Currently it does appear ISIS is exporting to PDS files one pixel off.

#3 Updated by Stuart Sides 10 months ago

  • Target version deleted (227)

#4 Updated by Tammy Becker 9 months ago

  • Target version set to 3.5.1 (Sprint 1)

#6 Updated by Stuart Sides 8 months ago

User test plans are done. These have been converted to bugs, so they will be resolved.

#7 Updated by Stuart Sides 7 months ago

  • Story points deleted (4)

#8 Updated by John Bonn 7 months ago

  • Assignee set to John Bonn

#9 Updated by John Bonn 7 months ago

  • Status changed from Acknowledged to In Progress

#10 Updated by John Bonn 7 months ago

  • Status changed from In Progress to Resolved

#11 Updated by Stuart Sides 5 months ago

  • Status changed from Resolved to Closed

Closed after testing by Trent

#12 Updated by Stuart Sides 4 months ago

  • Target version changed from 3.5.1 (Sprint 1) to 3.5.1 (2017-08-08 Aug)

Also available in: Atom PDF