Skip to content

Allow writing multi-model structure files with different atoms in each model#921

Merged
padix-key merged 1 commit into
biotite-dev:mainfrom
padix-key:issue-131
Jul 2, 2026
Merged

Allow writing multi-model structure files with different atoms in each model#921
padix-key merged 1 commit into
biotite-dev:mainfrom
padix-key:issue-131

Conversation

@padix-key

Copy link
Copy Markdown
Member

Fixes #131 .

@padix-key padix-key changed the title Allow writing multi-model structure files with different atoms in each model. Allow writing multi-model structure files with different atoms in each model Jun 30, 2026
@codspeed-hq

codspeed-hq Bot commented Jun 30, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 15.7%

⚡ 2 improved benchmarks
✅ 101 untouched benchmarks
⏩ 4 skipped benchmarks1

Performance Changes

Benchmark BASE HEAD Efficiency
benchmark_set_structure 10.8 ms 9.3 ms +16.21%
benchmark_set_structure_with_bonds 11.1 ms 9.6 ms +15.19%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing padix-key:issue-131 (efb271e) with main (9e9f6a1)

Open in CodSpeed

Footnotes

  1. 4 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@padix-key padix-key mentioned this pull request Jul 1, 2026
@padix-key padix-key force-pushed the issue-131 branch 2 times, most recently from 6c760dc to 19c3d2e Compare July 1, 2026 11:16
@padix-key padix-key merged commit 039fd62 into biotite-dev:main Jul 2, 2026
24 checks passed
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.

Support writing multiple AtomArray to structure file

1 participant