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 #11621 (new)

Opened 10 years ago

Last modified 8 years ago

Bug: Lack of clear codepath for image import

Reported by: bpindelski Owned by:
Priority: critical Milestone: Unscheduled
Component: Services Version: 5.0.0-beta1
Keywords: n.a. Cc: jamoore, jburel, mtbcarroll
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

The collaboration between Insight (especially OMEROGateway) and blitz (especially ImportLibrary) is overloaded with code repetition and unused methods. This ticket should capture any work needed to clean the code that is executed between those two classes.

Also - the OmeroJava integration test(-s) responsible for attempting an import are currently commented out.

Change History (6)

comment:2 Changed 10 years ago by jamoore

Please don't start a refactoring until #10325 is merged

comment:3 Changed 10 years ago by bpindelski

Josh: I wasn't planning to do it alone. This is a major chunk of work and definitely needs to involve more people.

comment:4 Changed 10 years ago by jamoore

  • Milestone changed from 5.0.0-beta2 to 5.0.0-beta3
  • Sprint OMERO 5 Beta 2 (1) deleted

Pushing to beta3. As discussed with J-m and Blazej, this is dependent on the import-set work. Perhaps after beta2 we work on a combined branch together to prevent conflict issues.

comment:5 Changed 9 years ago by jamoore

  • Milestone changed from 5.1.0 to 5.x

comment:6 Changed 8 years ago by jamoore

  • Milestone changed from 5.x to Unscheduled
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.64650 sec.)

We're Hiring!