Skip to content

Reports changes#7

Merged
CarlosBlancoGo merged 23 commits intomainfrom
report-changes
Mar 6, 2026
Merged

Reports changes#7
CarlosBlancoGo merged 23 commits intomainfrom
report-changes

Conversation

@CarlosBlancoGo
Copy link
Collaborator

No description provided.

CarlosBlancoGo and others added 23 commits February 23, 2026 11:37
- Remove r-plotly dependency. All plots now static ggplot
- Clean up redundant code, comments, and dead parameters
- Add meta-transcript body coverage profile plot
-Fixed operator precedence bug in count initialization to restore accurate FL-weighted transcript counts
-Fixed coordinate mismatch in build_meta_coverage_plot for ISM transcripts spanning introns to prevent valid transcripts from being dropped
…d fix PDF Transcript Classification table values sorting bug
…y basic features, and fix empty DataTables in hidden dropdown tabs
… the R script subprocess to support directory paths natively containing spaces.
…ce transcript body coverage by structural category plot
…utions to reads mode and change titles for clarity
… by isoform ID, using _count (FL) sum per (CB, junction_type) instead of groupby on comma-separated CB string
…rms mode

  - Fix genes_by_cb: explode CB/FL and sum FL per (CB, gene) instead of n()
  - Fix all junction-by-category blocks (PDF junc_aug, HTML junc_aug_html):
    replace broken join-by-CB+isoform with CB/FL explode + isoform-key-only join
  - Fix RT-switching junction figures (junc_rt): same CB/FL explode pattern;
    use sum(count[RTS_bool]) instead of sum(RTS_bool)
  - Fix mono-exonic figure: FL-explode cls_valid, use sum(count) instead of n()
  - Fix exon count bins by category: sum(count) instead of dplyr::n()
  - Fix exon count profile by category: sum(count) instead of dplyr::n()
  - Fix mean exons per category: FL-weighted mean (sum(exons*count)/sum(count))
    instead of unweighted median on non-exploded data
  - Replace complete(..., fill=list(...=0)) with NA fill for absent (CB, category)
    pairs across all junction/exon violin plots to eliminate zero-inflation
@CarlosBlancoGo CarlosBlancoGo merged commit 39646d8 into main Mar 6, 2026
2 checks passed
@CarlosBlancoGo CarlosBlancoGo deleted the report-changes branch March 6, 2026 15:37
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