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

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

OMERO.matlab: check version mismatch

Reported by: sbesson Owned by: sbesson
Priority: major Milestone: OMERO-4.4.7
Component: General Version: n.a.
Keywords: Matlab Cc: analysis@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: 2013-04-09 (7))

Description

See https://github.com/openmicroscopy/openmicroscopy/pull/811, unexpected errors may arise if using non-matching versions of OMERO.matlab and OMERO server.

Version mismatch should be checked when calling loadOmero(servername) and an error should be returned if versions are not compatible

Change History (4)

comment:1 Changed 6 years ago by sbesson

  • Status changed from new to accepted

comment:2 Changed 6 years ago by jburel

  • Sprint changed from 2013-03-12 (6)) to 2013-04-09 (7))

Moved from sprint 2013-03-12 (6))

comment:3 Changed 6 years ago by sbesson

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

Speaking with Jean-Marie, the way to read the server version is to use the Config Service on a session. This implies we would need to wrap the client.createSession(...) method and integrate a version check once the session is instantiated.

This change would certainly disrupt the typical workflow of the OMERO.matlab for the 4.4.x series. Marking this ticket as invalid but we may want to resurrect it if we move to a OO approach for 5.0.0 and beyond.

comment:4 Changed 6 years ago by jmoore

Sebastien/Jean?-Marie: if necessary we could push this into the createSession method itself, and then do the same for OmeroPy and OmeroCpp.

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

We're Hiring!