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

Opened 9 years ago

Closed 8 years ago

BUG: RFE Show error dialog if image to big for export

Reported by: dlindner Owned by: dlindner
Priority: minor Milestone: OMERO-5.2.0
Component: Insight Version: OMERO-5.1.3
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

See https://www.openmicroscopy.org/qa2/qa/feedback/11374/

Display a proper error message instead of crashing the application, if an image is too big for the export as ome.tiff

Change History (4)

comment:1 Changed 9 years ago by jamoore

  • Milestone changed from 5.2.0 to OMERO-5.2.0

Splitting due to milestone decoupling.

comment:2 Changed 9 years ago by dlindner

If the image is too big the 'export as ome-tif' option usually is not available. Unless it is a float or double image, see PixelsService?:

public boolean requiresPixelsPyramid(Pixels pixels) {
        String type = pixels.getPixelsType().getValue();
        if ("float".equals(type) || "double".equals(type))
            return false;
        final long sizeX = pixels.getSizeX();
        final long sizeY = pixels.getSizeY();
        final boolean requirePyramid = (sizeX * sizeY) > (sizes.getMaxPlaneWidth()*sizes.getMaxPlaneHeight());
        return requirePyramid;
    }

Actually a separate method isLargeImage() is required, if an image is too big for ome-tif can't be checked using the requiresPixelsPyramid() method.

comment:3 Changed 8 years ago by dlindner

Can be tackled in the scope of the new size limit configuration property, see discussion on https://github.com/openmicroscopy/openmicroscopy/pull/4223

comment:4 Changed 8 years ago by dlindner

  • Resolution set to fixed
  • Status changed from new 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.89757 sec.)

We're Hiring!