NOTICE! This is a static HTML version of a legacy ImageJ Trac ticket.

The ImageJ project now uses GitHub Issues for issue tracking.

Please file all new issues there.

Ticket #925 (assigned defect)

Opened 2012-02-01T09:24:13-06:00

Last modified 2013-06-07T15:55:34-05:00

Drawing tools too slow on big images

Reported by: bdezonia Owned by: curtis
Priority: major Milestone: imagej2-unscheduled
Component: Tools Version:
Severity: serious Keywords:
Cc: rob.vanthof@… Blocked By:
Blocking: #1170

Description

Open Embryos (a color image) and use the paint brush tool. The drawing lags behind quite a bit making the tool unusable.

Change History

comment:1 Changed 2012-02-01T09:25:36-06:00 by bdezonia

Note that it is related to image size. Embryos is very big. Drawing on clowns works fine.

comment:2 Changed 2012-02-01T09:38:29-06:00 by bdezonia

Tried making DrawingTool maintain multiple accessors (1 per channel) for color images. This had no effect and was abandoned.

comment:3 Changed 2012-02-01T09:50:10-06:00 by bdezonia

  • Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta2

Okay, color has nothing to do with it. Particles lags too. I think its a redraw speed issue. In AbstractLineTool() the event handler looks like this:

public void onMouseDrag(final MsDraggedEvent evt) {

if (drawingTool != null) {

drawingTool.lineTo(evt.getX(), evt.getY());
evt.getDisplay().getPanel().redraw();
evt.getDisplay().update();
evt.consume();

}

}

The redraw() and update() calls are slower for bigger images.

comment:4 Changed 2012-02-01T09:50:53-06:00 by bdezonia

  • Summary changed from Drawing tools too slow on color images to Drawing tools too slow on big images

comment:5 Changed 2012-05-14T10:47:52-05:00 by bdezonia

  • Blocking 1070 added

comment:6 Changed 2012-05-14T11:06:51-05:00 by bdezonia

  • Owner changed from bdezonia to curtis
  • Status changed from new to assigned
  • Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta4

Curtis, I think you have a better idea of how to fix this issue

comment:7 Changed 2012-06-08T10:46:27-05:00 by bdezonia

  • Blocking 1170 added; 1070 removed

comment:8 Changed 2012-07-17T11:50:02-05:00 by bdezonia

  • Cc rob.vanthof@… added

comment:9 Changed 2012-08-14T12:21:45-05:00 by bdezonia

  • Milestone changed from imagej-2.0.0-beta4 to imagej-2.0.0-beta5

Since we are going to work on display code during beta 5 I will move this issue there.

comment:10 Changed 2013-06-07T15:55:34-05:00 by curtis

  • Milestone changed from imagej2-b7-ndim-data to imagej2-unscheduled