Skip to content

Fix image paths and add dupRadar benchmark comparisons#15

Merged
ewels merged 2 commits into
mainfrom
claude/fix-rustqc-images-76bz1
Feb 14, 2026
Merged

Fix image paths and add dupRadar benchmark comparisons#15
ewels merged 2 commits into
mainfrom
claude/fix-rustqc-images-76bz1

Conversation

@ewels
Copy link
Copy Markdown
Member

@ewels ewels commented Feb 14, 2026

Summary

This PR improves documentation by fixing hardcoded image paths to use dynamic base URLs and adds visual comparisons of dupRadar outputs between R and RustQC implementations.

Key Changes

  • Fixed image path handling: Updated rseqc.mdx and dupradar.mdx to use dynamic base URL from import.meta.env.BASE_URL instead of hardcoded /RustQC/ paths. This ensures images load correctly regardless of deployment base path.

  • Added dupRadar benchmark comparisons: Expanded featurecounts.md with a new "Side-by-side plots" section that visually compares R dupRadar output against RustQC output for three plot types:

    • Density scatter plot
    • Duplication rate boxplot
    • Expression histogram
  • Improved documentation structure: Used CSS Grid layout for side-by-side image comparisons with proper spacing and captions, making it easier to verify that RustQC produces identical results to the reference R implementation.

Implementation Details

  • The dynamic base URL approach allows the documentation to work correctly in different deployment scenarios (root path vs. subdirectory)
  • Image comparisons use consistent styling with grid-template-columns: 1fr 1fr for equal-width columns
  • All new images reference the large benchmark dataset for comprehensive validation

https://claude.ai/code/session_01QMMExQTjDXR5jHHYQvr8NR

…ark plots

Use import.meta.env.BASE_URL instead of hardcoded /RustQC/ prefix for
image paths in outputs/dupradar.mdx and outputs/rseqc.mdx, matching
the pattern already used in index.mdx. This fixes broken images when
the site is deployed with a different base path (e.g., on Netlify).

Also add side-by-side dupRadar plot comparisons to the featureCounts
benchmark page, demonstrating that identical gene-level counts produce
identical downstream visualizations.

https://claude.ai/code/session_01QMMExQTjDXR5jHHYQvr8NR
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 14, 2026

Deploy Preview for rustqc ready!

Name Link
🔨 Latest commit 590c840
🔍 Latest deploy log https://app.netlify.com/projects/rustqc/deploys/699071361284c10008546c22
😎 Deploy Preview https://deploy-preview-15--rustqc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Convert dupradar.md and featurecounts.md to .mdx so image src attributes
can use import.meta.env.BASE_URL, ensuring images resolve correctly on
the deployed site regardless of the base path. Also convert inline style
strings to JSX objects as required by MDX.

https://claude.ai/code/session_01QMMExQTjDXR5jHHYQvr8NR
@ewels ewels merged commit d48ec7d into main Feb 14, 2026
4 of 8 checks passed
@ewels ewels deleted the claude/fix-rustqc-images-76bz1 branch February 14, 2026 13:16
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