Skip to content

SVGRenderer replacement and jbrowse version update#380

Merged
hextraza merged 14 commits intodiscvr-25.11from
25.11_fb_canvasRenderer
Feb 24, 2026
Merged

SVGRenderer replacement and jbrowse version update#380
hextraza merged 14 commits intodiscvr-25.11from
25.11_fb_canvasRenderer

Conversation

@hextraza
Copy link
Contributor

These files:

  • jbrowse/patches/@GMOD+bbi+8.1.1.patch
  • jbrowse/patches/@GMOD+bgzf-filehandle+6.0.12.patch

and the changes in jbrowse/src/client/JBrowse/Search/components/StandaloneSearchComponent.tsx

are temporary, pending a PR or two upstream. We need @jbrowse/components to export the fetchResults function. The @GMOD patches fix a weird interaction between our LabKey/Tomcat Content Security Policy headers and the way those packages do their WASM loading, which they just implemented for fast gunzipping.

The rest replaces ExtendedVariantRenderer with a default CanvasFeatureRenderer and moves all the relevant state for InfoFilters and activeSamples over to the ExtendedVariantDisplay. This required incrementing our jbrowse major version which had impacts all over the place. I'll move this out of draft status once I have an answer on some of the changes we need upstream and I have the JBrowse tests passing (most of the selenium stuff related to clicking on SNVs is broken.)

Copy link
Contributor

@bbimber bbimber left a comment

Choose a reason for hiding this comment

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

One minor comment only.

Sebastian Benjamin and others added 3 commits February 23, 2026 14:11
…tion or class'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@hextraza hextraza marked this pull request as ready for review February 24, 2026 17:43
@hextraza hextraza merged commit 9d3008c into discvr-25.11 Feb 24, 2026
8 checks passed
@hextraza hextraza deleted the 25.11_fb_canvasRenderer branch February 24, 2026 21:13
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.

2 participants