Skip to content

Fix logo on benchmarks website#7779

Merged
connortsui20 merged 1 commit intodevelopfrom
ct/fix-logo
May 4, 2026
Merged

Fix logo on benchmarks website#7779
connortsui20 merged 1 commit intodevelopfrom
ct/fix-logo

Conversation

@connortsui20
Copy link
Copy Markdown
Contributor

Summary

They were inconsistent with our current branding material. Only the black version changes on the current website.

Testing

N/A

@connortsui20 connortsui20 requested a review from lwwmanning May 4, 2026 15:43
@connortsui20 connortsui20 added the changelog/fix A bug fix label May 4, 2026
@connortsui20 connortsui20 changed the title Fix logo Fix logo on benchmarks website May 4, 2026
@connortsui20 connortsui20 marked this pull request as draft May 4, 2026 15:46
@connortsui20
Copy link
Copy Markdown
Contributor Author

actually it might be the case that both of these are wrong

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 4, 2026

Merging this PR will degrade performance by 24.95%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 1167 untouched benchmarks
⏩ 138 skipped benchmarks1

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

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation new_bp_prim_test_between[i64, 32768] 177.3 µs 236.2 µs -24.95%
Simulation bitwise_not_vortex_buffer_mut[128] 275.3 ns 246.1 ns +11.85%

Comparing ct/fix-logo (5eadf32) with develop (44a6367)

Open in CodSpeed

Footnotes

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

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@connortsui20
Copy link
Copy Markdown
Contributor Author

ok it is right now

@connortsui20 connortsui20 requested a review from mprammer May 4, 2026 16:06
@connortsui20 connortsui20 marked this pull request as ready for review May 4, 2026 17:40
@connortsui20 connortsui20 merged commit 2887cfb into develop May 4, 2026
65 of 66 checks passed
@connortsui20 connortsui20 deleted the ct/fix-logo branch May 4, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants