Project

General

Profile

Bug #4590

Unnecessary requirement for qview

Added by Stuart Sides 11 months ago. Updated 8 months ago.

Status:
Closed
Priority:
High
Assignee:
Marjorie Hahn
Category:
Applications
Target version:
Impact:

No expected impact.

Software Version:
Test Reviewer:

Description

If the data directory for ISIS is missing the templates area for base (i.e., $ISIS3DATA/base/templates). qview will give the following error when started:

I/O ERROR Cannot create AutoRegFactory. As a result, sub-pixel registration will not work.
I/O ERROR Unable to open [XXXXXXXXXXXXXXXXXx/isis/../data/base/templates/autoreg/qnetReg.def].


Related issues

Related to ISIS - Bug #4877: qnet - recent version of beta throws errors Closed

History

#1 Updated by Tammy Becker 11 months ago

  • Status changed from New to Acknowledged

#2 Updated by Stuart Sides 10 months ago

  • Status changed from Acknowledged to Assigned
  • Assignee set to Marjorie Hahn

#3 Updated by Marjorie Hahn 10 months ago

  • Status changed from Assigned to In Progress

#4 Updated by Tammy Becker 9 months ago

  • Target version set to 3.5.1 (Sprint 1)

#5 Updated by Marjorie Hahn 8 months ago

  • Status changed from In Progress to Resolved
  • Impact updated (diff)

#6 Updated by Ian Humphrey 8 months ago

  • Status changed from Resolved to Code Review

#7 Updated by Ian Humphrey 8 months ago

Looks good, good job on updating the code for coding standards too!

  • Make sure to add history comment with Fixes #4590 to ControlPointEdit.h

Suggestion/Question:

  • In registerPoint(), when the auto reg creation fails, should the p_autoRegFact be deleted?
    • If so, should this member also be deleted in ControlPointEdit's destructor?

#8 Updated by Stuart Sides 8 months ago

  • Status changed from Code Review to Closed

#9 Updated by Stuart Sides 8 months ago

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

#10 Updated by Lynn Weller 7 months ago

  • Related to Bug #4877: qnet - recent version of beta throws errors added

Also available in: Atom PDF