Skip to content

feat: Support custom contig sep characters in pairix index#398

Merged
nvictus merged 3 commits intoopen2c:masterfrom
nvictus:feat-pairix-custom-block-char
Mar 8, 2024
Merged

feat: Support custom contig sep characters in pairix index#398
nvictus merged 3 commits intoopen2c:masterfrom
nvictus:feat-pairix-custom-block-char

Conversation

@nvictus
Copy link
Copy Markdown
Member

@nvictus nvictus commented Mar 8, 2024

By default, the | character is used to delimit contig names in pairix index blocks, e.g. chr1|chr1, chr1|chr2, etc.

However, many FASTA sequence names contain pipes, which conflicts with this usage. Pairix supports creating an index with a different contig separator using -w, but this hasn't been supported in cooler cload pairix. See #234.

This PR lets the user provide a custom separator to cooler cload pairix via --block-char if it happens to be known or is discovered by the user (unfortunately, pypairix provides no way to determine this character dynamically) and will error if the separator is incorrect.

@nvictus nvictus requested a review from GarrettNg March 8, 2024 03:17
@nvictus nvictus merged commit b9d2e8a into open2c:master Mar 8, 2024
@nvictus nvictus mentioned this pull request Mar 8, 2024
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