Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Bump iviewer 0.16.0 #445

Merged
merged 1 commit into from
Feb 17, 2025
Merged

Bump iviewer 0.16.0 #445

merged 1 commit into from
Feb 17, 2025

Conversation

will-moore
Copy link
Member

Frances was hoping to get this in the next release. Is it possible to deploy onto idr-next?

cc @dominikl

@sbesson sbesson self-requested a review February 13, 2025 14:11
Copy link
Member

@sbesson sbesson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deployed by running idr-omero-web.yml followed by idr-omero-readonly.yml (see #436 for the rationale) against prod126. Can you check this works as expected?

@sbesson sbesson requested review from dominikl and francesw February 13, 2025 16:38
@francesw
Copy link
Member

I'm experiencing a caching issue with some images. But with non-cached images, it is working as expected.

@dominikl
Copy link
Member

dominikl commented Feb 14, 2025

Tested with a few images from idr0163, activing more channels. Sometimes I get "Failed to load tiles", but sometimes I get the proper error message. Is that what you meant with caching issues too Frances?
Sorry, ignore my comment. I didn't use private window, in private browser all looks good!

@will-moore
Copy link
Member Author

Seems to be working fine for me too.

@francesw
Copy link
Member

I tried in a private window, but I still get the old error message of "Failed to load tiles" with some images. But, yes, I think this is a caching issue.

@sbesson
Copy link
Member

sbesson commented Feb 17, 2025

Assuming the issues reported above are unrelated to this PR, merging

@sbesson sbesson merged commit 5e43ad3 into IDR:master Feb 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants