Project

General

Profile

Feature #5258

ControlNetVersioner refactor

Added by Jesse Mapel 4 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
-
Impact:

New binary control network files will not be compatible with old ISIS versions. New Pvl control network files will still be compatible with old ISIS versions.

Memory usage when reading and writing control networks will be reduced by about 30%.

The Choosername and DateTime keywords will not be present in the ControlPoint objects in new Pvl control network files unless they are explicitly set. See #5282 for more information.

Software Version:
Test Reviewer:

Description

In order to make improvements in memory usage when reading and writing control network files, ControlNetVersioner needs to be refactored.

Currently, it reads an entire control network file into memory and creates the actual ControlNet structure. This consumes a large amount of RAM during reading and writing (about 50% more memory than the ControlNet structure uses). Instead, the control network file should be loaded in pieces so that the entire file does not need to be duplicated in memory.

This requires changing how ControlNetVersioner works and how ControlNet, ControlPoint, and ControlMeasure interact with it.

History

#1 Updated by Jesse Mapel 4 months ago

Kristin is not the only person working on this task. The full software development team is:

Jeannie Backer
Kristin Berry
Adam Goins
Jesse Mapel
Stuart Sides

Work is being done in the CNetImprovements_ControlNetVersioner branch of the ISIS3 repository.

#2 Updated by Jesse Mapel 4 months ago

  • Status changed from New to In Progress

#3 Updated by Kristin Berry 3 months ago

  • Status changed from In Progress to Resolved

#4 Updated by Jesse Mapel 3 months ago

  • Impact updated (diff)

#5 Updated by Adam Goins 3 months ago

  • Impact updated (diff)

#6 Updated by Kristin Berry 3 months ago

  • Impact updated (diff)

#7 Updated by Kristin Berry 3 months ago

  • Impact updated (diff)

#8 Updated by Kristin Berry 3 months ago

  • Impact updated (diff)

#9 Updated by Kristin Berry 3 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF