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

Opened 10 years ago

Closed 10 years ago

Bug: Review of thumbnail caching and loading strategy

Reported by: cxallan Owned by: wmoore
Priority: critical Milestone: OMERO-Beta4.3.2
Component: Web Version: n.a.
Keywords: n.a. Cc: atarkowska, cneves
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: 2011-07-21 (1)

Description

OMERO.web makes use of the single thumbnail loading mechanisms of the thumbnail service. When there are rendering settings changes behind the scenes of OMERO.web (or with multiple tabs, windows or logins; case in point the image viewer), due to its usage of the same thumbnail service and a lack of close() calls thumbnails can be rendered with stale state.

We should make an overarching review of the thumbnail load semantics in these multi-user and multi-view contexts so that no stale state is used. Furthermore, due to browser caching semantics it will be important to ensure that the "right" thumbnails are cached by the browser.

Change History (6)

comment:1 Changed 10 years ago by atarkowska

  • Cc atarkowska added; wmoore removed
  • Owner changed from atarkowska to wmoore

Once this is resolved #6100 should be fixed.

comment:2 Changed 10 years ago by wmoore

  • Sprint set to 2011-07-21 (1)

comment:3 Changed 10 years ago by wmoore

  • Status changed from new to accepted

comment:4 Changed 10 years ago by wmoore

Problem with closing all thumbnailstores in ImageWrapper.saveDefaults() is that you get Ice Object Not Exist exceptions from the other thumbnail store proxy object wrappers.
After discussion with Chris and Ola:
Better to simply close TB proxy at the end of each getThumbnail().

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

(In [7b09f46d5a8fdb0ce290c2f82cfc0332e0f75a6a/ome.git] on branch develop) ImageWrapper?.getThumbnail() closes TB store proxy every time. See #6116

comment:6 Changed 10 years ago by wmoore

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

Seems that recent changes to webgateway/views.py d8c06ecf52cc9fccaf403b43fcd2970a77f955e5 has fixed caching of thumbnails and cache is cleared when any changes are made to rendering settings.

Only issue now is to set a default location for cache #6314.

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

We're Hiring!