-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update ATAC-seq DM to receive common report file #502
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 258287 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 258241 lines exceeds the maximum allowed for the inline comments feature.
Code Climate has analyzed commit 8011d32 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 93.8% (0.0% change). View more on Code Climate. |
💯 |
What
Moves the report file to the preamble section of the template so it is expected batch-level instead of sample-level.
Actual changes.
Why
To be compatible with GD2CAR ATAC-seq data
Checklist
Please include and complete the following checklist. You can mark an item as complete with the
- [x]
prefix:black
andflake8
have been used to ensure styling guidelines are metdocs/
has be regenerated and README has been updated to explain major changes & new featurescidc_schemas/__init__.py