Skip to content

Add Base - Level Reporting - Requirement 4#123

Merged
beichen1024 merged 8 commits intomainfrom
bc_req4
Oct 7, 2025
Merged

Add Base - Level Reporting - Requirement 4#123
beichen1024 merged 8 commits intomainfrom
bc_req4

Conversation

@beichen1024 beichen1024 marked this pull request as ready for review October 7, 2025 14:27
@colossalpat colossalpat requested a review from Copilot October 7, 2025 14:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a "base" column to various aggregation tables and reports, which contains the sum of mapped lengths for alignment data. The change enhances the reporting functionality by providing both count and length-based metrics.

  • Adds base column calculation using pre-aggregated alignment data
  • Updates R Markdown report to display the new base counts in tables
  • Modifies flip-flop aggregation to include length summation alongside counts

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/aggregate_tables.py Adds base column calculation by merging alignment length data and updating aggregation functions
src/report.Rmd Updates table headers and column selections to include the new "Base Counts" column

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@vijayformbio vijayformbio changed the title req4 Add Base-Level Reporting - Requirement 4 Oct 7, 2025
@vijayformbio vijayformbio changed the title Add Base-Level Reporting - Requirement 4 Add Base - Level Reporting - Requirement 4 Oct 7, 2025
@beichen1024 beichen1024 merged commit f770aae into main Oct 7, 2025
8 checks passed
@beichen1024 beichen1024 deleted the bc_req4 branch October 7, 2025 17:12
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