Skip to content

Conversation

@pnbabu
Copy link
Contributor

@pnbabu pnbabu commented Sep 24, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Sep 24, 2025

🐰 Bencher Report

Branch1249/merge
Testbedubuntu-latest

🚨 1 Alert

IterationBenchmarkMeasure
Units
ViewBenchmark Result
(Result Δ%)
Upper Boundary
(Limit %)
0tests/nest_continuous_benchmarking/test_nest_continuous_benchmarking.py::TestNESTContinuousBenchmarking::test_stdp_nn_synapseLatency
seconds (s)
📈 plot
🚷 threshold
🚨 alert (🔔)
3.94 s
(+14.92%)Baseline: 3.43 s
3.77 s
(104.48%)

Click to view all benchmark results
BenchmarkLatencyBenchmark Result
seconds (s)
(Result Δ%)
Upper Boundary
seconds (s)
(Limit %)
tests/nest_continuous_benchmarking/test_nest_continuous_benchmarking.py::TestNESTContinuousBenchmarking::test_stdp_nn_synapse📈 view plot
🚷 view threshold
🚨 view alert (🔔)
3.94 s
(+14.92%)Baseline: 3.43 s
3.77 s
(104.48%)

BenchmarkLatencyBenchmark Result
seconds (s)
(Result Δ%)
Upper Boundary
seconds (s)
(Limit %)
tests/nest_continuous_benchmarking/test_nest_continuous_benchmarking.py::TestNESTContinuousBenchmarking::test_stdp_nn_synapse📈 view plot
🚷 view threshold
🚨 view alert (🔔)
3.23 s
(-5.58%)Baseline: 3.42 s
3.76 s
(85.84%)

BenchmarkLatencyBenchmark Result
seconds (s)
(Result Δ%)
Upper Boundary
seconds (s)
(Limit %)
tests/nest_continuous_benchmarking/test_nest_continuous_benchmarking.py::TestNESTContinuousBenchmarking::test_stdp_nn_synapse📈 view plot
🚷 view threshold
🚨 view alert (🔔)
3.21 s
(-6.00%)Baseline: 3.42 s
3.76 s
(85.45%)

🐰 View full continuous benchmarking report in Bencher

Copy link
Contributor

@clinssen clinssen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking care of this! I have a few comments and suggestions.

@clinssen clinssen linked an issue Sep 25, 2025 that may be closed by this pull request
@clinssen
Copy link
Contributor

clinssen commented Oct 2, 2025

Thanks very much for taking care of this! Could we change the version check for more future-proofing? I'm otherwise cool with this PR, feel free to merge it once this little change is in and the CI passes.

@clinssen clinssen merged commit ed9f540 into nest:master Oct 10, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong logging of NEST version

2 participants