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.

Sign up for GitHub

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

Jump to bottom

util/alignment/async: Add async indexed reader #296

Open
zaeleus opened this issue Aug 26, 2024 · 0 comments
Open

util/alignment/async: Add async indexed reader #296

zaeleus opened this issue Aug 26, 2024 · 0 comments
Labels
enhancement New feature or request util

Comments

@zaeleus
Copy link
Owner

zaeleus commented Aug 26, 2024

I wonder if it also makes sense to add an asynchronous version of an indexed reader? Given the implementations on SAM/BAM/CRAM are !Send I guess we probably need that auto trait before we could do that? And I hope I understand this correctly, we might also want Sync as currently it is !Sync, though my understanding is it doesn't have to be Sync to be used in async code correct?

Originally posted by @mbhall88 in #286 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request util
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zaeleus and others