Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #4393 (closed)

Opened 13 years ago

Closed 13 years ago

ROI polygon vv polyline vv line

Reported by: wmoore Owned by: ajpatterson
Priority: minor Milestone: OMERO-Beta4.3
Component: Model Version: n.a.
Keywords: n.a. Cc: jburel, dzmacdonald, jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description (last modified by jburel)

Summary

  • Model supports Polyline and Line
  • Insight supports Polygon and Polyline
  • Database and omero.model support Line, Polyline and Polygon.

Potential problems: Eventual export of ROI polygon. Client writes a 'Line' - user views it in Insight but then can't read a Line from database.

Polygon

The model http://www.openmicroscopy.org/Schemas/ROI/2010-06/ROI.xsd does not mention Polygon but there is an omero.model.polygonI and Insight writes Polygon to the database to describe a closed line. See #4317

Line

Line is supported in the model and database, and Insight will read a line (x1,y1, x2,y2) but always save it as a PolyLine with a points string. PolyLine is supported in the model, although points string differs from model. See #4392


Decision

  • Schema: No change.
  • Insight: replace usage of Polygon by closed PolyLine. #4838
  • Insight: Do not convert Line to PolyLine. #4839
  • DB: Remove Polygon, replace by closed PolyLine. #4840

Change History (5)

comment:1 Changed 13 years ago by jburel

  • Cc jburel dzmacdonald added

comment:2 Changed 13 years ago by jburel

  • Cc jmoore added
  • Description modified (diff)

comment:3 Changed 13 years ago by jburel

  • Description modified (diff)

comment:4 Changed 13 years ago by jburel

Referencing ticket #4317 has changed sprint.

comment:5 Changed 13 years ago by jburel

  • Resolution set to fixed
  • Status changed from new to closed

Tickets have been created so closing this one.

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.72947 sec.)

We're Hiring!