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 6 years ago

Closed 6 years ago

Last modified 6 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 6 years ago by mtbcarroll

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

comment:2 Changed 6 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 6 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 6 years ago by mtbcarroll

What if some of the images in SEPARATED are orphaned?

comment:5 Changed 6 years ago by jamoore

Then I would expect images to be returned as with findContainerHierarchies

comment:6 Changed 6 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 6 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 6 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 6 years ago by jamoore

  • Owner mtbcarroll deleted
  • Status changed from accepted to new

comment:10 Changed 6 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 6 years ago by mtbcarroll

  • Owner set to mtbcarroll

comment:12 Changed 6 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 6 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 6 years ago by mtbcarroll

  • Status changed from new to accepted

comment:15 Changed 6 years ago by mtbcarroll

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

comment:16 Changed 6 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 6 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 6 years ago by wmoore

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

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

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

comment:20 Changed 6 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 6 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 6 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 6 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.85476 sec.)

We're Hiring!