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 #10782 (closed)

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

MIF queries

Reported by: jamoore Owned by: mtbcarroll
Priority: major Milestone: 5.0.0-beta1
Component: Services Version: n.a.
Keywords: n.a. Cc: fs@…, wmoore, cxallan
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: FS Demo 4.3

Description

In order to implement the proper delete/chgrp actions described in #10780 et al., it may be easier to detect server-side which images have siblings in a separate container and provide this information to the clients.

Logically, this means:

  • pass in a container (e.g. Dataset id)
  • find all the images in the dataset (CONTAINED)
  • remove those which are either:
    • not in a MIF, or
    • the entire MIF is in the container
  • call this set SEPARATED
  • find other containers of the same type that contain images from SEPARATED.

The proper name for this type of query is still unclear. "find separated siblings" might work. This is related to findContainerHierarchy and loadContainerHierarchies.

Change History (23)

comment:1 Changed 11 years ago by mtbcarroll

  • Owner set to mtbcarroll
  • Status changed from new to accepted

comment:2 Changed 11 years ago by mtbcarroll

Perhaps this new method should be offered alongside those listed in IContainer.ice and implemented by PojosImpl.java.

comment:3 Changed 11 years ago by jburel

actually adjusting some calls from the existing methods in Icontainer will be required so the graph was be nicely displayed in client

comment:4 Changed 11 years ago by mtbcarroll

What if some of the images in SEPARATED are orphaned?

comment:5 Changed 11 years ago by jamoore

Then I would expect images to be returned as with findContainerHierarchies

comment:6 Changed 11 years ago by mtbcarroll

Ah, return a set with both containers and images in it? (Basically, a container if a separated image is in it, and an image if the image is orphaned?)

comment:7 Changed 11 years ago by jamoore

Right. findContainerHierarchies returns a collection of either Projects Datasets or Images. The highest one in the hierachy possible.

comment:8 Changed 11 years ago by wmoore

What I'm doing in the web just now when chgrp fails is to show the user how they attempted to 'split' each fileset: see https://github.com/openmicroscopy/openmicroscopy/pull/1138

If I were to provide this same UI before they hit the chgrp "OK" button, then I'd like to know which filesets would be split, and which images would be in/out.

E.g.

filesets = listSplitFilesets("Dataset", [123, 456])          # Trying to move 2 datasets

#  { 1: { true: [3, 4, 5],  false: [6, 7, 8] },   11: { true: [ 20, 21], false: [ 22 ] } }

In the example, 2 filesets (1 & 11) would have been split if I tried to chgrp 2 datasets. Image IDs are true: (those inside the 2 Datasets) and false: Not included in Datasets.

NB: If a fileset is split between the 2 Datasets, it should not be returned by this query.

comment:9 Changed 11 years ago by jamoore

  • Owner mtbcarroll deleted
  • Status changed from accepted to new

comment:10 Changed 11 years ago by jamoore

  • Sprint changed from FS demo 4.1 to FS demo 4.2

Wasn't needed during FS demo 4.1

comment:11 Changed 11 years ago by mtbcarroll

  • Owner set to mtbcarroll

comment:12 Changed 11 years ago by mtbcarroll

Would it be enough to know which images are missing (need to be added to avoid split filesets) or do we actually need an excitingly structured return value?

comment:13 Changed 11 years ago by wmoore

I'm just saying that this is what we're currently displaying to users in the web UI. If you don't give me the missing AND included image IDs then I've got to load the filesets and their images to get that data, which is possible but not as nice.

comment:14 Changed 11 years ago by mtbcarroll

  • Status changed from new to accepted

comment:15 Changed 11 years ago by mtbcarroll

I'll indeed try to provide something that is pretty much what you suggest then.

comment:16 Changed 11 years ago by jburel

  • Sprint changed from FS demo 4.2 to FS Demo 4.3

Moved from sprint FS demo 4.2

comment:17 Changed 11 years ago by mtbcarroll

fixed by https://github.com/openmicroscopy/openmicroscopy/pull/1214 but leaving open so that the accompanying integration tests aren't forgotten

comment:18 Changed 11 years ago by wmoore

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

comment:19 Changed 11 years ago by Will Moore <will@…>

(In [9f269cd9ac0f1051c71709869ed39bcba9f6712d/ome.git] on branch develop) Various fileset query tests. See #10782

comment:20 Changed 11 years ago by Josh Moore <josh@…>

(In [424a167af0fc9957cbc0e7c02d2f3cb6797a0c06/ome.git] on branch develop) Merge pull request #1217 from will-moore/split_fileset_query_tests_10782

Various fileset query tests. See #10782

comment:21 Changed 11 years ago by mtbcarroll

The accompanying integration tests I had in mind in comment 17 are now in https://github.com/mtbc/openmicroscopy/commit/67629c8fe6d62cfc8b393e85a928ce8ef6034e39 and more will follow from #11013.

comment:22 Changed 11 years ago by Mark Carroll <m.t.b.carroll@…>

  • Remaining Time set to 0

(In [d1c4ec7580f40171f92b3aee1f440f78742a1368/ome.git] on branch develop) fix #10782: add IContainer.getImagesBySplitFilesets

comment:23 Changed 11 years ago by Josh Moore <josh@…>

(In [ba6e48595b876c67d99a247eb8d115184a86c12e/ome.git] on branch develop) Merge pull request #1214 from mtbc/trac-10782

fix #10782: add IContainer.getImagesBySplitFilesets

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

We're Hiring!