Skip to content

Use biongff vizarr#509

Draft
lubianat wants to merge 3 commits into
ome:mainfrom
lubianat:use-biongff-vizarr
Draft

Use biongff vizarr#509
lubianat wants to merge 3 commits into
ome:mainfrom
lubianat:use-biongff-vizarr

Conversation

@lubianat
Copy link
Copy Markdown
Contributor

@lubianat lubianat commented May 1, 2026 •
edited
Loading

  • Changes the URL for the Vizarr embedding
  • Changes the ~500GB 2D fish for a ~200GB 4D brain (~185, summing up all scales via validator)

Supersedes #508

lorenzocerrone and others added 2 commits May 1, 2026 11:51
@lorenzocerrone
fix ome#507
46441a4
@lubianat
Use biongff fork of Vizarr
a5b4348 
* fixes ome#507, supersedes ome#508 (closes ome#508)

* changes the fish for the brain, adapts the text
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Automated Review URLs

@lubianat
Re-add Lorenzo's <div>
58853cb 
(for some reason, build of the new image is not fitting well natively)
@lubianat lubianat marked this pull request as draft May 1, 2026 13:40
@lubianat
Copy link
Copy Markdown
Contributor Author

lubianat commented May 1, 2026

Screenshot_20260501_154517_Firefox.jpg

It does not work on my phone, but it was not working before either. It works on other phones. Likely a different bug.

@joshmoore joshmoore marked this pull request as ready for review May 1, 2026 13:54
@lorenzocerrone
Copy link
Copy Markdown

lorenzocerrone commented May 4, 2026

It does not work on my phone, but it was not working before either. It works on other phones. Likely a different bug.

I can confirm, both this PR version and the main version do not work on my phone (iPhone).

@lubianat lubianat mentioned this pull request May 12, 2026
Merged
@lubianat lubianat marked this pull request as draft May 12, 2026 18:07
@joshmoore
Copy link
Copy Markdown
Member

joshmoore commented May 13, 2026

A few things:

  • Note the change of the submodules
  • Generally supportive of using the new URL wherever we do embed 👍
  • Might either @-mention the team here if there's not an open issue
  • Moving forward with Remove the Vizarr display #511 for the moment though

joshmoore pushed a commit that referenced this pull request May 13, 2026
@lubianat @joshmoore
Remove the Vizarr display
fe6e3ba 
Reverting #483 while a robust fix for #507 is not found.

The fix on #509  is only partial; it also fails for some mobile devices (as noted in #507)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lubianat @lorenzocerrone @joshmoore