Skip to content

Conversation

@sdvillal
Copy link
Contributor

@sdvillal sdvillal commented Dec 30, 2025

@padix-key
Copy link
Member

Thanks for finding this 👍.

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 31, 2025

CodSpeed Performance Report

Merging #850 will degrade performance by 13.58%

Comparing sdvillal:fix-numpy-reshape-compat (3f70f84) with main (693a9cb)

Summary

⚡ 1 improvement
❌ 1 (👁 1) regression
✅ 61 untouched

Benchmarks breakdown

Benchmark BASE HEAD Efficiency
👁 benchmark_superimpose[superimpose_homologs] 24.4 ms 28.3 ms -13.58%
benchmark_get_assembly 900.8 ms 663.9 ms +35.67%

@padix-key
Copy link
Member

The changes in performance are probably simply caused by the newer NumPy version that is installed in the benchmark job now.

@padix-key padix-key merged commit 4651068 into biotite-dev:main Jan 1, 2026
28 checks passed
@sdvillal
Copy link
Contributor Author

sdvillal commented Jan 1, 2026

Thanks for the quick turnaround @padix-key, and happy new year!

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