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

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

omero.gateway more general purpose

Reported by: wmoore Owned by: wmoore
Priority: minor Milestone: OMERO-Beta4.3
Component: OmeroPy Version: n.a.
Keywords: n.a. Cc: cneves, atarkowska, jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-01-13 (3)

Description (last modified by wmoore)

Some assumptions reduce the utility of omero.gateway for other web apps.

Will make a list of examples here and we can review. E.g.

  • BlitzObjectWrapper.save() and .saveAs() both return None (instead of saved object) :TODO Docs - this is OK
  • BlitzObjectWrapper._linkAnnotation (self, ann) tries to manually set permissions. Doesn't work with 4.2 #3801
  • _BlitzGateway.clone() TODO: Add anonymous and userAgent parameters from parent.
  • ImageWrapper._getProjectedThumbnail (self, size, pos) Can't see any projection code here? TODO: Docs to say this is 'internal' method and setProjection is called first.
  • getAnnotation() add link=rv[0]
  • Image & Dataset assumed single parent #3802
    • BlitzOjectWrapper?._moveLink might be more useful if it didn't assume only 1 parent - option allowed you to specify the oldParent.
    • listParents (self, single=True, withlinks=False) would be more useful in Django templates if default single=False, allowing you to always treat the result as iterable.
    • getAncestry() assumes single parent. Only used for cache keys?
    • ImageWrapper.getProject() and getDataset() return None if Image is in more than one Dataset.


Change History (13)

comment:1 Changed 13 years ago by wmoore

  • Description modified (diff)

comment:2 Changed 13 years ago by jmoore

  • Cc jmoore added

comment:3 Changed 13 years ago by wmoore

  • Status changed from new to assigned

comment:4 Changed 13 years ago by wmoore

  • Owner changed from wmoore to cneves-x
  • Status changed from assigned to new

Carlos - maybe some of these can be fixed right away - others might need clarification and discussion. Cheers.

comment:5 Changed 13 years ago by wmoore

(In [8741]) Editing some doc comments in omero.gateway. See #3655, #3656

comment:6 Changed 13 years ago by jburel

  • Sprint changed from 2010-12-09 (21) to 2010-12-23 (22)

Moved from sprint 2010-12-09 (21)

comment:7 Changed 13 years ago by jburel

  • Sprint changed from 2010-12-23 (22) to 2011-01-13 (23)

Moved from sprint 2010-12-23 (22)

comment:8 Changed 13 years ago by wmoore

  • Description modified (diff)
  • Owner changed from cneves-x to wmoore

comment:9 Changed 13 years ago by wmoore

  • Description modified (diff)

comment:10 Changed 13 years ago by wmoore

  • Status changed from new to accepted

comment:11 Changed 13 years ago by wmoore

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

(In [8888/omero]) Docs and small fixes to gateway.init.py. Closes #3656

comment:12 Changed 13 years ago by Carlos Neves <carlos@…>

(In [72cc23e9f4d65ae8a31e542b5abffc9c69ca3f26/ome.git] on branch dev_4_1_custom) Docs and small fixes to gateway.init.py. Closes #3656

git-svn-id: file:///home/svn/omero/trunk@8888 05709c45-44f0-0310-885b-81a1db45b4a6

comment:13 Changed 13 years ago by Carlos Neves <carlos@…>

(In [72cc23e9f4d65ae8a31e542b5abffc9c69ca3f26/ome.git]on branches 4_1_custom, dev_4_1_custom, josh/4.1-web-backport) Docs and small fixes to gateway.init.py. Closes #3656

git-svn-id: file:///home/svn/omero/trunk@8888 05709c45-44f0-0310-885b-81a1db45b4a6

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

We're Hiring!