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

Opened 11 years ago

Last modified 8 years ago

Refactor ImportCandidates into Bio-Formats

Reported by: jamoore Owned by:
Priority: major Milestone: Unscheduled
Component: General Version: 4.4.9
Keywords: fs Cc: fs@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

The ImportCandidates classes in ome.formats.importer are completely independent of the rest of the importer logic and so could be refactored into Bio-Formats. One possible name for this class would be DirectoryReader. The idea is that a directory could be passed to setId and then the various filesets could be pieced apart. A further extension which would make it more like ImportCandidates would be to take an array of paths (whether directories or files) and perform the same operation.

Change History (5)

comment:1 Changed 11 years ago by jamoore

  • Keywords fs added

comment:2 Changed 11 years ago by mlinkert

  • Owner set to mlinkert

comment:3 Changed 10 years ago by jburel

  • Milestone changed from OMERO-5 to 5.0.0
  • Version set to 4.4.9

comment:4 Changed 8 years ago by jamoore

  • Milestone changed from 5.x to Unscheduled

comment:5 Changed 8 years ago by mlinkert

  • Owner mlinkert deleted

Note that getFiles(...) in loci.tests.testng.TestTools? does something very similar (as part of the full repo tests). Long term it would be nice if both the tests and ImportCandidates could use the same logic for building lists of filesets.

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.65517 sec.)

We're Hiring!