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

Opened 16 years ago

Closed 14 years ago

LoadContainerhierarchy

Reported by: jburel Owned by: jburel
Priority: major Milestone: OMERO-Beta4.2
Component: Services Version: 3.0-M3
Keywords: n.a. Cc: dzmacdonald
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-05-27 (10)

Description

When the root node is a Dataset,
even if the option is set to no leaves
the images are loaded.

Change History (13)

comment:1 Changed 16 years ago by jmoore

  • Status changed from new to assigned

This ticket is related to #882 . When I added the workaround:

            Set<Image> images = new HashSet<Image>();
            for (IObject o : l) {
                Dataset d = (Dataset) o;
                images.addAll(d.linkedImageList());
            }
            if (images.size() > 0) {
                iQuery.findAllByQuery(loadCountsImages, new Parameters()
                        .addSet("list", images));
            }

the images were automatically being loaded. I can manually delete them if you like.

comment:2 Changed 16 years ago by jmoore

r2319 has a failing test.

comment:3 Changed 16 years ago by jburel

The images are loaded but not the pixels.
If the noleaves is set to true, the images should not be loaded.

I now have a workaround in insight but I think Ola was planning
on using some of the methods exposed by the Pojos service.

comment:4 Changed 16 years ago by jburel

Sorry READ noleaves = true, Images loaded.
(and not not loaded as written in previous comment)

comment:5 Changed 16 years ago by jburel

ignore last comment, body is up but brain still in bed!

comment:6 Changed 16 years ago by jmoore

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

r2320 should correct. Closing, but please report any weirdness using loadContainerHierarchy with rootNodeType==Dataset.class and noLeaves.

comment:7 Changed 15 years ago by dzmacdonald

  • Cc dzmacdonald added
  • Resolution fixed deleted
  • Status changed from closed to reopened

This call loads all images when dataset is root node in loadContainerHierarchy even when parameter == noleaves.

This is a call used in gatewayService.getDatasets()

comment:8 Changed 15 years ago by jmoore

  • Milestone changed from 3.0-Beta3 to OMERO-Beta4.0.1

comment:9 Changed 15 years ago by jmoore

  • Owner changed from jmoore to dzmacdonald
  • Status changed from reopened to new

Donald, re-assigning to you for determining where it should be properly moved. Feel free to re-assign it to me when the milestone is clear.

comment:10 Changed 15 years ago by dzmacdonald

  • Milestone changed from OMERO-Beta4.0.1 to OMERO-Beta4.1

comment:11 Changed 14 years ago by jburel

  • Milestone changed from Unscheduled to OMERO-Beta4.2
  • Owner changed from dzmacdonald to jburel
  • Sprint set to 2010-05-27 (10)

Add test to check if this is the case

comment:12 Changed 14 years ago by jburel

  • Status changed from new to assigned

comment:13 Changed 14 years ago by jburel

  • Remaining Time set to 0
  • Resolution set to fixed
  • Status changed from assigned to closed

(In [7240]) Added test to load dataset w/o images (Closed #907)

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

We're Hiring!