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

imgData JSON with no stateful services #548

Closed
wants to merge 2 commits into from

Conversation

will-moore
Copy link
Member

@will-moore will-moore commented Apr 9, 2024

This PR is purely to allow deployment of the #536 on idr-testing by rebasing those changes onto an older version of omero-web that supports django-32. To make this easier I have now consolidated those changes into a single commit.

See #537 which shows the history of code changes (based on django-32 omero-web), including various commits used for A/B/C performance testing, which was performed with that branch deployed on idr-testing.

Deployed on idr-testing...

for server in omeroreadwrite omeroreadonly-1 omeroreadonly-2 omeroreadonly-3 omeroreadonly-4; do ssh $server "sudo /opt/omero/web/venv3/bin/pip uninstall -y omero-web && sudo /opt/omero/web/venv3/bin/pip install git+https://github.com/ome/omero-web.git@refs/pull/548/head && sudo service omero-web restart"; done

Use ?resolution_levels=false

load all imgData without state, except zoomLevels

Use owners or create rdef if None exist

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Don't require canAnnotate() to try create rdef

flake8 fix

Don't load RE for preview panel rdef

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Don't lookup resolution levels for small images

Remove TEMP A/B testing

Revert changes to rendering engine usage for detecting pyramids

Don't bail if re fails to load - still try to load rdef, marshal channels etc
@will-moore will-moore marked this pull request as draft April 9, 2024 14:54
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.

1 participant