Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collapse introns #4033

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Collapse introns #4033

wants to merge 2 commits into from

Conversation

cmdcolin
Copy link
Collaborator

@cmdcolin cmdcolin commented Nov 2, 2023

This is a work in progress for collapsing introns in the linear genome view

Very basic implementation, filtering exon subfeatures and adds a small window around each exon, and creates a list of displayedRegions from that

fixes #557

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Nov 2, 2023
@cmdcolin cmdcolin added enhancement New feature or request and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Nov 2, 2023
@cmdcolin cmdcolin added the size large pretty big label Nov 22, 2023
@cmdcolin cmdcolin marked this pull request as draft November 22, 2023 16:42
@cmdcolin
Copy link
Collaborator Author

it was mentioned recently that this could be made an external plugin while the details of this are getting resolved. there are some tricky issues but if it's external, little lower bar for acceptance

Copy link

codecov bot commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 62.53%. Comparing base (a097560) to head (9158297).
Report is 132 commits behind head on main.

Current head 9158297 differs from pull request most recent head 5c93cdc

Please upload reports for the commit 5c93cdc to get more accurate results.

Files Patch % Lines
...aseLinearDisplay/models/BaseLinearDisplayModel.tsx 0.00% 18 Missing ⚠️
...earPileupDisplay/SharedLinearPileupDisplayMixin.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4033      +/-   ##
==========================================
- Coverage   62.61%   62.53%   -0.08%     
==========================================
  Files        1088     1088              
  Lines       31495    31518      +23     
  Branches     7531     7532       +1     
==========================================
- Hits        19721    19711      -10     
- Misses      11602    11632      +30     
- Partials      172      175       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cmdcolin cmdcolin force-pushed the collapse_introns branch 2 times, most recently from 9254752 to 5c93cdc Compare June 18, 2024 03:17
Misc

Update snaps

Collapse intron p1

Properly size linear genome view after creation

Bump deps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size large pretty big
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create method to collapse introns in LGV
1 participant