Ticket #102 (closed defect: fixed)

Opened 13 years ago

Last modified 13 years ago

OverlayNodedObject getData() method sometimes makes invalid Gridded2DSets

Reported by: sorber Owned by: sorber
Priority: major Milestone:
Component: visbio Severity:
Keywords: overlays Cc:
Blocked By: Blocking:

Description

Occasionally but rarely get a VisADSetException when drawing ONOs. 99% of the time there's no such exception. Double check math; if it seems okay, try to reproduce the error (maybe using an automated test).

Change History

comment:1 in reply to: ↑ description Changed 13 years ago by sorber

One reason this occurs: the OverlayObject.isDrawing flag is false when the user resumes drawing an existing polyline, so OverlayTransform tries to draw a grid. Each time the user places a node, the last two nodes are briefly overlapping, resulting in an invalid grid. To solve: go through polyline tool and add setDrawing() calls where appropriate.

comment:2 Changed 13 years ago by sorber

  • Priority changed from critical to major

comment:3 in reply to: ↑ description Changed 13 years ago by sorber

Replying to sorber:

Occasionally but rarely get a VisADSetException when drawing ONOs. 99% of the time there's no such exception. Double check math; if it seems okay, try to reproduce the error (maybe using an automated test).

The title of this bug should read (OverlayUtil).getSelectionLayer(), not getData()

comment:4 Changed 13 years ago by sorber

OverlayUtil.getNodedLayer() no longer tries to create a Gridded2DSet for pairs of colocational nodes.

comment:5 Changed 13 years ago by sorber

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.