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

DRAFT - ignore me. #1940

Closed
wants to merge 3 commits into from
Closed

DRAFT - ignore me. #1940

wants to merge 3 commits into from

Conversation

cmnbroad
Copy link
Contributor

Test a fix for Takuto's branch.

cmnbroad and others added 3 commits January 16, 2024 12:21
…mentCollection implementations to use nio paths.

Change CleanSam to use getReferencePath.

preliminary changes

test passing for now

CreateSequenceDictionary looks good

working on downsamplesam

temporary as I fix the issue

remove 2

just a commit

near ready

another checkpoint

checkpoint

all good except maybe the cram

ready

chris comments no 1

review edits 1

done proofreading

chris edit 2

PicardHtsPath -> IOPath where appropriate

another edit

another round of edits

louis

update htsjdk

chris suggestion replaceExtension

Co-authored-by: Chris Norman <cnorman@broadinstitute.org>

chris edit

reorganize

proofread

manually resolve conflicts

khalid conflict that was not caught

some fix

fixed more fails
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