Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Rename module: sv-analysis -> chromothripsis #1069

Merged
merged 3 commits into from
May 13, 2021

Conversation

LauraEgolf
Copy link
Contributor

@LauraEgolf LauraEgolf commented May 13, 2021

This is an initial pull request to rename the sv-analysis module and prepare it for updates, as discussed in #1007 (comment).

The following changes were made:

  • Renamed module "sv-analysis" -> "chromothripsis" to reflect the module's main purpose (running ShatterSeek and classifying chromothripsis regions)
  • Updated module README.md
  • Updated entry in analyses/README.md table to reflect name change
  • Updated config.yml to reflect name change where the module's two scripts were previously entered (Let me know if there's a different way you'd prefer to do this!)
  • Removed existing results, since new results will be produced

Note: 02-shatterseek.R will be modified in the next PR. (And I will fill out the PR template to provide details)

- Rename "sv-analysis" to "chromothripsis" to reflect the module's main purpose (running ShatterSeek and classifying chromothripsis regions)
- Update module README.md
- Update entry in analyses/README.md table
- Remove existing results, since these will be updated
I changed the module name for the module's two scripts where they were previously entered into config.yml (Let me know if there's a different way you'd prefer to do this!)
@jaclyn-taroni jaclyn-taroni self-requested a review May 13, 2021 10:12
Copy link
Member

@jaclyn-taroni jaclyn-taroni left a comment

Choose a reason for hiding this comment

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

This looks great @LauraEgolf, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants