Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

don't attempt ROI stats on big images #223

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

will-moore
Copy link
Member

@will-moore will-moore commented Sep 13, 2024

If you try to run the Batch_ROI_Export script on Big images, it fails with:

> > serverStackTrace = ome.conditions.ApiUsageException: This method cannot handle tiled images yet. 
> > at ome.services.roi.GeomTool.getStatsRestricted(GeomTool.java:415) 

To avoid this, we first check for whether this is a big image and if so, we don't try to load stats.

To test, draw ROIs on a Big Image(s) and run Batch_ROI_Export script:

Screenshot 2024-09-13 at 09 10 48

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/omero-web-script-batch-roi-export/103609/5

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/omero-web-script-batch-roi-export/103609/7

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants