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

Added a section about the Arima mapping pipeline #70

Merged
merged 1 commit into from
Jun 7, 2024
Merged

Added a section about the Arima mapping pipeline #70

merged 1 commit into from
Jun 7, 2024

Conversation

muffato
Copy link
Member

@muffato muffato commented Jun 7, 2024

No description provided.

@muffato muffato requested a review from jgrg June 7, 2024 08:18
@muffato muffato self-assigned this Jun 7, 2024
Copy link

@jgrg jgrg left a comment

Choose a reason for hiding this comment

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

I think it should be "Arima Genomics provide us" rather than "provides us". Actually "supply" might be better:

Arima Genomics, who supply the kits we use to produce Hi-C data for genome assembly, have released open source scripts which aid mapping the reads to a genome assembly. We have modified them to support the CRAM files produced by Sanger's sequencing teams, and have made these changes are available in our fork of the repository on GitHub.

(Sorry, I couldn't resist fiddling with the wording!)

@muffato muffato merged commit 1ef9fc6 into main Jun 7, 2024
3 checks passed
@muffato muffato deleted the arima branch June 7, 2024 10:10
@muffato
Copy link
Member Author

muffato commented Jun 7, 2024

Sorry, I accidentally pushed to main and closed this PR.

I added your changes as a commit d91e13c

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