Skip to content

[PR] 进行了一点更新#43

Open
pzweuj wants to merge 18 commits into
shiquan:masterfrom
pzweuj:master
Open

[PR] 进行了一点更新#43
pzweuj wants to merge 18 commits into
shiquan:masterfrom
pzweuj:master

Conversation

@pzweuj

@pzweuj pzweuj commented Dec 24, 2025

Copy link
Copy Markdown

@shiquan

shiquan commented Jan 2, 2026

Copy link
Copy Markdown
Owner

Thank you for your contribution! I will benchmark and update some of these changes in the main repo.
Some comments:

  • I/O缓冲优化这步没有太多必要,因为bgzf_write这个函数已经包含了一个数据缓存池。
  • 多线程这部分的思考可以再讨论下,因为bgzf.[ch]本身wrap进去了一个多线程调度,性能瓶颈除了IO,最大的消耗是pack和unpack BAM文件。更“完美”的方案是一个线程负责IO,其余工作线程负责unpack BAM records->统计处理。

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