Project

General

Profile

Bug #4772

cnetcombinept doesn't properly set a prior lat/lon to the adjusted values

Added by Kris Becker 8 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Applications
Target version:
Impact:

This will not have an impact on ISIS, since the fix was made in the application itself and not in an object.

Software Version:
Test Reviewer:

Description

The SETAPRIORIBEST option in cnetcombinept is intended to reset the a priori latitude and longitude values to the best available values. The best values are typically the adjusted lat/lon values produced by jigsaw.

The wrong ControlPoint method is invoked in the main to set these values correctly.


Related issues

Related to ISIS - Bug #4869: cnetcombinept doesn't properly combine control points that contain the same control measures in each point (i.e., duplicates) Closed
Related to ISIS - Bug #4870: cnetcombinept should set merged control measure types to Candidate so that pointreg can be run efficiently Closed

History

#1 Updated by Tammy Becker 8 months ago

  • Status changed from New to Acknowledged

#2 Updated by Stuart Sides 7 months ago

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

#3 Updated by Kris Becker 7 months ago

  • Related to Bug #4869: cnetcombinept doesn't properly combine control points that contain the same control measures in each point (i.e., duplicates) added

#4 Updated by Kris Becker 7 months ago

  • Related to Bug #4870: cnetcombinept should set merged control measure types to Candidate so that pointreg can be run efficiently added

#6 Updated by Stuart Sides 6 months ago

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

#7 Updated by Tyler Wilson 6 months ago

  • Assignee set to Tyler Wilson

#8 Updated by Tyler Wilson 6 months ago

  • Status changed from Acknowledged to In Progress

#9 Updated by Tyler Wilson 6 months ago

  • Status changed from In Progress to Resolved

#10 Updated by Tyler Wilson 6 months ago

  • Impact updated (diff)

#11 Updated by Tyler Wilson 6 months ago

To make things easier for testing, I have combined all three of the cnetcombinept tickets into one ticket which can be tested by pointing your ISIS version here:

/work/projects/isis/latest/m04870_4772_4869/isis/

The other two tickets are:
https://isis.astrogeology.usgs.gov/fixit/issues/4870
https://isis.astrogeology.usgs.gov/fixit/issues/4869

#12 Updated by Tammy Becker 6 months ago

I have tested this and it looks like the adjusted lat, lon, radius are copied to the apriori lat, lon, radius fields within the network as expected.

Test area:
/work/users/tbecker/Cassini/ISS_Enceladus/CDAP_ISS_Enceladus/MikeWork/global-network/MrgWithSouthPole/Salih/NewImgs_Orb250/
network_test_m04772_cnetcombinept.csh

#13 Updated by Tammy Becker 6 months ago

  • Status changed from Resolved to Closed

#14 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