Skip to content

Pass limit through Python Vortex#6239

Merged
gatesn merged 1 commit intodevelopfrom
ngates/python-limit
Feb 2, 2026
Merged

Pass limit through Python Vortex#6239
gatesn merged 1 commit intodevelopfrom
ngates/python-limit

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Feb 2, 2026

And fix up limit to be a u64

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn added the changelog/feature A new feature label Feb 2, 2026
@gatesn gatesn enabled auto-merge (squash) February 2, 2026 02:46
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 2, 2026

Merging this PR will degrade performance by 33.04%

❌ 1 regressed benchmark
✅ 1137 untouched benchmarks
⏩ 1384 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation chunked_bool_into_canonical[(1000, 10)] 43.1 µs 64.3 µs -33.04%

Comparing ngates/python-limit (246759a) with develop (26fa799)2

Open in CodSpeed

Footnotes

  1. 1384 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.

  2. No successful run was found on develop (2f9b92c) during the generation of this report, so 26fa799 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@gatesn gatesn merged commit 2c198c4 into develop Feb 2, 2026
42 of 46 checks passed
@gatesn gatesn deleted the ngates/python-limit branch February 2, 2026 02:51
AdamGS pushed a commit that referenced this pull request Feb 2, 2026
And fix up limit to be a u64

Signed-off-by: Nicholas Gates <nick@nickgates.com>
danking pushed a commit that referenced this pull request Feb 6, 2026
And fix up limit to be a u64

Signed-off-by: Nicholas Gates <nick@nickgates.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants