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

Opened 8 years ago

Closed 5 years ago

Projection of Z-range

Reported by: wmoore Owned by: wmoore
Priority: major Milestone: 5.1.0
Component: OmeroPy Version: n.a.
Keywords: n.a. Cc: python-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description

Maybe not for 4.3? but we need to support Z-start, Z-stop and stepping in the Z-projection offered by imageWrapper.

For example, some scripts (Split_View_Figure.py) cannot use Blitz Gateway for image rendering because they need to handle Z-projection #4376.

Carlos - if you have an idea how you'd like this implemented, let me know and I can do the work.

E.g.
Add these attributes to imageWrapper

image._zstart
image._zend
image._zstepping

and methods to support this workflow

image.setProjection('intmax')
image.setZstart(5)
image.setZend(10)
image.setStepping(2)   # 1 by default
jpg = image.renderJpeg(z, t)

Change History (9)

comment:1 Changed 8 years ago by cneves

  • Owner changed from cneves-x to wmoore
  • Remaining Time set to 0.2

Projection code lives in ImageWrapper? currently, the only option being processed is projection type.

The way I would implement this would be to replace the _pr property of ImageWrapper?, which is just a string, and turn it into a dictionary with all the needed projection variables, followed by adding the needed optional arguments to ImageWrapper?.setProjection() with the current hard coded values as the defaults.

ImageWrapper?.getProjection() might be kept as is (returns the projection type as a string), changed to return a copy of the dictionary holding all the variables or have the method name changed to getProjectionType to avoid ambiguity.
Just remember that this is also used in webgateway, so you'll need to grep and replace if you change the method name or return value.

ImageWrapper?.getProjections() (which returns a list of possible projection keys as strings) is not used at all, and can be removed for now unless you plan to use it.

I'm giving this back to you but feel free to kick it back if you want me to go ahead with the proposed changes.

comment:2 Changed 8 years ago by jmoore

  • Milestone changed from OMERO-Beta4.3 to Unscheduled

Closed #4278 and moved to #5926 (unscheduled)

comment:3 Changed 7 years ago by jmoore

Referencing ticket #9092 has changed sprint.

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

(In [f16fbdd77d38f3f110da2685ff577b609c784352/ome.git] on branch develop) Add Z-projection range to Blitz Gateway. See #5150

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

(In [fb4ea60763e113d448893642877fb61ed6a07440/ome.git] on branch develop) Add Z-projection range to webgateway. See #5150

comment:7 Changed 6 years ago by wmoore

  • Milestone changed from Unscheduled to 5.0.0
  • Status changed from new to accepted
  • Version 4.4.10 deleted

comment:8 Changed 6 years ago by wmoore

  • Milestone changed from 5.0.0 to 5.1.0

comment:9 Changed 5 years ago by wmoore

  • Remaining Time changed from 0.2 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed
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.123518 sec.)

We're Hiring!