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

Allow tabix indexing of GAF graph alignment files #1763

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

adamnovak
Copy link
Contributor

This pulls in @jmonlong's implementation of Tabix indexing for GAF files, in graph node ID space.

This is to allow random access to bgzip-compressed GAF files in vgteam/vg using a tabix index.

commit 1d11645
Merge: 5426f5c 6a7d33a
Author: Adam Novak <anovak@soe.ucsc.edu>
Date:   Mon Mar 18 14:30:53 2024 -0400

    Merge remote-tracking branch 'origin/master' into bgzip-gaf

commit 5426f5c
Author: Jean Monlong <jean.monlong@gmail.com>
Date:   Wed Sep 20 12:09:05 2023 +0200

    update usage doc

commit bdf5717
Author: Jean Monlong <jmonlong@ucsc.edu>
Date:   Thu Mar 23 23:43:53 2023 -0700

    finish merge

commit f35cf3e
Merge: 3ad8d7f c1634e7
Author: Jean Monlong <jmonlong@ucsc.edu>
Date:   Thu Mar 23 23:16:43 2023 -0700

    Merge branch 'develop' into bgzip-gaf

commit 3ad8d7f
Author: Jean Monlong <jmonlong@ucsc.edu>
Date:   Sun Oct 16 09:39:24 2022 -0400

    index gaf with smallest and largest nodes
@adamnovak
Copy link
Contributor Author

@whitwham Github closed #1759 without permission when I pushed the wrong thing trying to do the rebase. This is the rebased version you requested.

htslib/tbx.h Outdated Show resolved Hide resolved
@whitwham whitwham self-assigned this Mar 25, 2024
@adamnovak
Copy link
Contributor Author

@whitwham I've fixed the duplicate line.

@whitwham whitwham merged commit 3cfe876 into samtools:develop Mar 26, 2024
9 checks passed
@whitwham
Copy link
Contributor

Merged. Thank you.

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