899f9da8129add8c64c86b405c87b3a97738a2ad
lrnassar
  Fri May 1 17:12:18 2026 -0700
Update mpraVarDb description page and trackDb stanza per QA pass. refs #37359

Description page (mpraVarDb.html):
- Primary citation corrected from a fabricated "Wang T, Matreyek KA,
Yang X." entry to Jin et al., 2024 (PMID 39325859, the actual published
Bioinformatics paper for MPRAVarDB).
- References for all 18 source studies added.
- Studies-table counts corrected to match the data: Tewhey 27,138 to 23,430
(the biggest gap, ~14%), Griesemer 72,588 to 72,546, Abell 29,582 to
29,564, Mouri 14,551 to 14,549, McAfee 10,310 to 10,302, Cooper 5,340
to 5,330, Lu 1,038 to 1,036.
- Color counts on Display Conventions corrected to actual post-liftOver
values: 22,451 dark red / 17,773 orange / 198,804 grey = 239,028.
- "Note on study type" paragraph clarifying that ~41% of variants
(Griesemer + Schuster 3'UTR) measure post-transcriptional regulation
rather than transcription.
- "Cell-type specificity" reminder, "Note on Kircher 2019" saturation-
mutagenesis structure, "Interpreting log2FC" subsection with the
linear-fold-change translation, and "Significance thresholds across
studies" Methods paragraph noting that source studies use a mix of
FDR thresholds (0.05/0.10) and that Kircher reports raw regression
p-values rather than FDR.
- Kircher element list reworded to avoid double-counting UC88 as both
a named element and "an ultraconserved enhancer".

trackDb stanza (mpra.ra, mpraVarDb subtrack):
- filterValues.cellLine fixed: PC3 was hiding 26,546 Schuster rows (data
has "PC3 cell"), SF7996 was hiding 3,896 Kircher GBM rows (data has
bundled "HEK293T,SF7996"), and SK-MEL-28 (1,510) and K562+GATA1 (31)
were missing entirely.  Added comma-escape syntax (HEK293T,,SF7996)
per slNameListFromCommaEscaped.  Recovers ~31,983 hidden rows.
- HTML mouseOver with bold field names and line breaks.
- urls rsid="https://www.ncbi.nlm.nih.gov/snp/$$" added so rsIDs are
clickable linkouts to dbSNP.
- dataVersion "MPRAVarDB snapshot 2026-03-10" added.

Makedoc (mpra.txt): QA review note added with the trackDb-only fixes
and the items punted to upstream / next rebuild for Tao Wang.

diff --git src/hg/makeDb/doc/hg38/mpra.txt src/hg/makeDb/doc/hg38/mpra.txt
index ee09f197ef3..ed45ad17294 100644
--- src/hg/makeDb/doc/hg38/mpra.txt
+++ src/hg/makeDb/doc/hg38/mpra.txt
@@ -1,117 +1,163 @@
 # MPRA superTrack (hg38) - Redmine #37359
 # -----------------------------------------------------------------------------
 # Two subtracks: mprabase (MPRA Base enhancer elements) and mpraVarDb (MPRA-tested
 # regulatory variants).  trackDb stanzas live in human/hg38/mpra.ra.  Description
 # pages: mpra.html, mprabase.html, mpraVarDb.html.
 
 # =============================================================================
 # mprabase subtrack - max Mar 30 2026
 # =============================================================================
 # No local processing. The bigBed was provided directly by Varda Singhal
 # (Ahituv Lab, UCSF) via UCSC hubspace and dropped into the gbdb path.
 #
 # Source (upstream bigBed):
 #   https://genome.ucsc.edu/hubspace/72/Varda006/Varda_Final_Hub/final_authorPMID.mean_v2.bb
 # Full upstream hub:
 #   https://genome.ucsc.edu/hubspace/72/Varda006/Varda_Final_Hub/hub.txt
 # Upstream SQLite sits alongside the bigBed:
 #   /hive/data/genomes/hg38/bed/mpra/mprabase/mprabase_v4_9.3.db
 # That DB corresponds to MPRA Base v4.9.3 and is the source of truth for
 # reproducing the bigBed if Varda ever refreshes the upstream hub.
 
 mkdir -p /hive/data/genomes/hg38/bed/mpra/mprabase
 cd /hive/data/genomes/hg38/bed/mpra/mprabase
 wget https://genome.ucsc.edu/hubspace/72/Varda006/Varda_Final_Hub/final_authorPMID.mean_v2.bb -O mprabase.bb
 
 # gbdb symlink:
 #   /gbdb/hg38/mpra/mprabase/mprabase.bb -> /hive/data/genomes/hg38/bed/mpra/mprabase/mprabase.bb
 
 # Historical note: an earlier attempt lifted from hg19 via a custom SQLite
 # liftover table (hg38CustomLiftover.RDS, preserved in the build dir), but
 # had one feature beyond chrom size.  Replaced by the pre-built hub file
 # above, so the liftOver path is not used.
 
 # =============================================================================
 # mpraVarDB subtrack - max Mar 10 2026 (claude/max), QA rebuild Apr 21 2026 (lou)
 # =============================================================================
 # Source:
 #   https://mpravardb.rc.ufl.edu/ (UFL web server)
 # Snapshot date: Mar 10 2026 (CSV via the "download_all" endpoint).  The
 # MPRAVarDB project does not publish version numbers; track the snapshot
 # date and the session URL together as the provenance pair.
 #
 # Input CSV contains 242,818 variants from 18 MPRA studies, with coordinates
 # in either hg19 or hg38: 213,689 hg19, 29,129 hg38, 3,676 with NA coords.
 # Script liftOvers hg19 -> hg38, merges with native hg38, and emits bigBed9+13.
 
 mkdir -p /hive/data/genomes/hg38/bed/mpra/mpravardb
 cd /hive/data/genomes/hg38/bed/mpra/mpravardb
 wget 'https://mpravardb.rc.ufl.edu/session/27d7af46df917aed91f4cca7bee378a2/download/download_all?w=' -O mpravardb.csv
 
 # Convert, liftOver, merge, and build bigBed.  Output: mpravardb.bb (239,028 rows).
 python3 ~/kent/src/hg/makeDb/scripts/mpravardb/mpravardbToBed.py
 
 # gbdb symlink:
 #   /gbdb/hg38/mpra/mpravardb/mpravardb.bb -> /hive/data/genomes/hg38/bed/mpra/mpravardb/mpravardb.bb
 
 # -----------------------------------------------------------------------------
 # QA rebuild Apr 21 2026 (RM #37359)
 # -----------------------------------------------------------------------------
 # mpravardbToBed.py updated to:
 #   - sanitize UTF-8 in user-visible string fields (curly quotes, primes,
 #     NBSP mojibake) before writing BED.  Prior build had ~246k non-ASCII
 #     byte occurrences across 100,961 rows (42% of track) including mangled
 #     rsIDs like "rs34425335NBSP-MOJIBAKE".
 #   - pval_to_score() now returns 0 (not 1000) for non-positive / out-of-range
 #     pvalue.  Prior build gave score=1000 to ~7,400 rows whose upstream pvalue
 #     was literal 0 (mostly NA-coded-as-0), inflating those to the top of any
 #     score-sorted view.
 #   - safe_float() now returns NaN (was 0.0) for NA / empty / non-numeric
 #     upstream values.  27,065 rows whose upstream pvalue was literal "NA"
 #     now store pvalue="nan" instead of "0.0", so untested variants no longer
 #     masquerade as p=0 in the details page and are excluded by the default
 #     filter.fdr / filter.log2FC range sliders.  bedToBigBed accepts the
 #     literal string "nan" in float fields.
 #
 # Pre-rebuild backup preserved at:
 #   /hive/data/genomes/hg38/bed/mpra/mpravardb/mpravardb.bb.preQA-backup
 #
 # Reproduce QA rebuild:
 #   cd /hive/data/genomes/hg38/bed/mpra/mpravardb
 #   python3 ~/kent/src/hg/makeDb/scripts/mpravardb/mpravardbToBed.py
 
 # =============================================================================
 # Known outstanding items (see RM #37359)
 # =============================================================================
 # - mprabase rebuild items to fold into Varda's next bigBed:
 #     * Mattioli 2020 reference field starts with "musculus ..." (species word
 #       merged into title upstream).  Varda confirmed 2026-04-23 she will fix.
 #     * AutoSQL percentile_rank description currently says "Percentile rank
 #       within cell line"; the data is actually computed per (cell_line, assay,
 #       PMID) experiment.  Fix the .as comment to "Percentile rank within
 #       experiment" so the schema page matches the description page.
 #     * Element-name disambiguation: HepG2-XX%-LM and similar auto-generated
 #       names collide across Inoue 2017 and Klein 2020 because both reused the
 #       same ENCODE-derived 171 bp library and produced the same percentile.
 #       Surface: 149 of 625 unique names are reused across multiple PMIDs;
 #       4 are exact (chrom,start,end,name) duplicates.  Encode PMID or short
 #       study tag in the name to disambiguate.
 # - mprabase chr14:69999387-69999388 (HeLa STARR-seq, PMID 23328393, Arnold 2013)
 #   was previously flagged as an orphan.  Varda confirmed (2026-04-23) it is
 #   valid: HeLa was a proof-of-concept in an otherwise Drosophila STARR-seq
 #   paper (Stark Lab).  Row added to the experiments table in mprabase.html.
 # - Klein et al. 2020 (PMID 33046894) is an MPRA-design benchmarking paper
 #   that ran the same 2,440-element library through nine different assays.
 #   The track has three Klein 2020 sub-rows (lentiMPRA, plasmidMPRA, STARR-seq);
 #   confirm with Varda which underlying sub-designs MPRA Base pulled, since
 #   the Klein 2020 authors flag HSS as the worst-correlated of the nine and
 #   recommend pGL4 / ORI / 5'/5' WT.  Description page can be sharpened once
 #   confirmed.
 # - mpraVarDB preserves ~42k (chrom,start,end,name) duplicate rows (same rsID
 #   tested in multiple cells/studies).  Users disambiguate via the
 #   filterValues.cellLine / filterValues.mpraStudy filters in the trackDb.
 # - ~7,400 rows have upstream pvalue=0 and fdr=0 (not NA).  Could be genuine
 #   precision-floor significance or an upstream "not tested" encoding; the
 #   distinction is not recoverable from the CSV.  With pval_to_score returning
 #   0 for p<=0, these no longer dominate score-sorted views but their details
 #   page still reads "pvalue: 0.0".  Upstream clarification needed.
+#
+# QA review 2026-05-01 (RM #37359, Lou):
+#   Found and fixed in trackDb only (no bigBed rebuild this round):
+#   - filterValues.cellLine had four broken entries hiding ~31,983 rows
+#     (13% of track): PC3 vs PC3 cell mismatch (26,546 rows), SF7996 needed
+#     comma-escape syntax for the bundled HEK293T,,SF7996 data value
+#     (3,896 rows), missing SK-MEL-28 (1,510) and K562+GATA1 (31).  All
+#     four corrected; filter now matches all 32 distinct cellLine values
+#     in the data.
+#   - Description page references rebuilt for all 18 source studies plus
+#     the corrected primary citation (Jin et al. 2024, PMID 39325859 in
+#     Bioinformatics; the previous "Wang T, Matreyek KA, Yang X." citation
+#     was fabricated -- not the actual authors of either the preprint
+#     PMID 38617248 or the published paper).
+#   - 7 studies-table row counts corrected to match data (Tewhey, Griesemer,
+#     Abell, Mouri, McAfee, Cooper, Lu).
+#   - HTML mouseOver upgraded to bold/multi-line.
+#   - dataVersion "MPRAVarDB snapshot 2026-03-10" added to stanza.
+#   - urls rsid="https://www.ncbi.nlm.nih.gov/snp/$$" added so rsIDs are
+#     clickable linkouts.
+#   - Methods + Display Conventions paragraphs added: scoring methodology
+#     differs across studies, post-transcriptional vs transcriptional
+#     distinction (Griesemer/Schuster 3'UTR), Kircher saturation
+#     mutagenesis structure, log2FC interpretation.
+#
+#   Punted to Redmine for Max / Tao Wang (potential next-rebuild items):
+#   - 5,092 rows (Mouri, Tewhey) have pvalue > 1 (impossible; max 8.96).
+#     FDR appears valid; pvalue field looks like a t-statistic mislabeled.
+#     mouseOver/details show misleading p-values for ~2% of significant
+#     rows.
+#   - 30,921 rows display literal "nan" / "None" in mouseOver and
+#     details fields where NA-coded values were preserved verbatim.  Two
+#     sentinel conventions coexist ("None" in 53k eQTL rows, "NA" in 2k
+#     Kircher rows).
+#   - Upstream typos in 28,810 rows: "30 UTR" in 26,546 Schuster
+#     descriptions, "Familial hypercholesterol emia" in 2,176 Kircher
+#     diseases, "Alchol use disorder" in 88 Rao diseases.
+#   - 60,860 rows have description="GWAS" (no detail) -- upstream limit.
+#   - 1,069 rows have multi-allelic alt collapsed into one row (e.g.
+#     "T/A,G") with one log2FC/pvalue.
+#   - 2,088 rows preserve hg19-coord-style names (e.g. "1_1403972_CG")
+#     post-liftOver to hg38; coordinates and name no longer match.
+#   - 969 rows are colored red (FDR<0.05) but pvalue=nan -- mouseOver
+#     reads "FDR: 0.001 / p-value: nan" which looks contradictory.
+#   - 250-char truncation in mpravardbToBed.py cuts Griesemer
+#     descriptions mid-sentence; should raise/remove the cap.