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

feat: html and excel depth table for relevant positions #14

Closed
wants to merge 1 commit into from

Conversation

chels0
Copy link

@chels0 chels0 commented Dec 14, 2023

This PR:

(If this is a release PR, no need to add following. Leave this part empty)
(Use the following lines to create a PR text body. Make sure to remove all non-relevant one after you're done)
(Repeat each field as many times as necessary)

Added: for new features.
Changed: for changes in existing functionality.
Deprecated: for soon-to-be removed features.
Removed: for now removed features.
Fixed: for any bug fixes.
Security: in case of vulnerabilities.

Review and tests:

  • Tests pass
  • If you've fixed a bug or added code that should be tested, add tests!
  • Code review
  • CHANGELOG.md is updated
  • New code is executed and covered by tests, and test approve
  • Module compatibility section in README.md is up to date

@chels0 chels0 requested review from pyrevo and ljmesi as code owners December 14, 2023 15:11
Copy link
Member

Choose a reason for hiding this comment

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

Did the chr id shifted from UCSC to Ensembl?

Copy link
Author

Choose a reason for hiding this comment

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

The reason why I changed it is because the target_rsid.bed file in the data folder is in Ensembl format. Don't know why the target_rsid file in the integration test was different. One rule (append_id_to_gdf) hasn't been working properly in the integration test before

Copy link
Member

@pyrevo pyrevo left a comment

Choose a reason for hiding this comment

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

Hej it looks great, nicely done!

I just commented about that chromosome that changed from 'chr6' to '6' but I guess it depends on the bam file you were using.

@chels0 chels0 closed this Dec 14, 2023
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