Skip to content

feat(viewer): open any finding in the genome viewer from the live site or the row itself - #4

Closed
geiresx wants to merge 4 commits into
mainfrom
pgp/live-viewer
Closed

geiresx wants to merge 4 commits into
mainfrom
pgp/live-viewer

Conversation

@geiresx

@geiresx geiresx commented Nov 4, 2026

Copy link
Copy Markdown
Owner

The report linked only the ClinVar hits, the secondary findings and the
mitochondrial variants to the genome viewer, as one line of links under
each table. Every findings table now ends with a Viewer column, one link
per row to that row's position: ClinVar hits, secondary findings, mito
variants, deep intronic sites, the cDNA scan, AnnotSV class 4-5 and the
calls moved to class 3, slivar's pairs in trans and mosaic alterations.
A column of its own keeps every other cell's text as it was.

Bead: pgp-awpe
The genome viewer ran only on 127.0.0.1 of the machine holding the data.
bin/owngenome_live_viewer.py serves the same igv.js page for every run of
the live site, behind its password, and points the served report's Viewer
links at it. The igv.js pin moves to versions.env so both launchers share
it. tests/test_live_viewer.py checks the login, the policy, the byte
ranges, the refused files, the report links, the access log and the page
in headless Chrome; tests/test_viewer.py checks every findings table and
reads the pin from versions.env.

Bead: pgp-5bja.8
Bead: pgp-awpe
…ps tracks with an index out of GENOME_DIR and serves quoted names

The live viewer compared the still-quoted request name with the file name, so a
reference named with a space, '#' or '?' answered 404. It now unquotes it. An
index symlinked out of GENOME_DIR was served; now its track, file and index are
left out. Each new check has a planted mistake that must fail it.
@geiresx geiresx closed this Nov 5, 2026
@geiresx
geiresx deleted the pgp/live-viewer branch November 5, 2026 08:55
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