feat: Comment on PR and auto update comment #223
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
* create PR comment when PR is available
* add gh token to workflow
* create review instead of review comment
* disable unnecessary ci runs
* add pr write permissions
* remove pending review
ktrz
force-pushed
the
feat/98-autoupdate-pr-comment
branch
from
d64c4b6 to
0ee988e
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
* revert removing pending review
* add logic for updating comment
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Go Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
BenchmarkFib10 |
310.3 ns/op |
310.7 ns/op |
1.00 |
BenchmarkFib20 |
39602 ns/op |
39528 ns/op |
1.00 |
BenchmarkFib20WithAuxMetric |
39878 ns/op 4.000 auxMetricUnits |
39638 ns/op 4.000 auxMetricUnits |
1.01 |
BenchmarkFib20WithAuxMetric - ns/op |
39878 ns/op |
39638 ns/op |
1.01 |
BenchmarkFib20WithAuxMetric - auxMetricUnits |
4 auxMetricUnits |
4 auxMetricUnits |
1 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Benchmark.js Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
fib(10) |
1666682 ops/sec (±0.18%) |
1614900 ops/sec (±0.33%) |
0.97 |
fib(20) |
13521 ops/sec (±0.24%) |
13365 ops/sec (±0.17%) |
0.99 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Python Benchmark with pytest-benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
bench.py::test_fib_10 |
76639.10714283153 iter/sec (stddev: 8.032532335846265e-7) |
76322.25409653576 iter/sec (stddev: 0.0000010082750379909067) |
1.00 |
bench.py::test_fib_20 |
615.4022089304406 iter/sec (stddev: 0.000012812298584847541) |
620.0468031518758 iter/sec (stddev: 0.00001294268552372628) |
1.01 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
C++ Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
fib_10 |
158.5494359313285 ns/iter |
158.5773734900016 ns/iter |
1.00 |
fib_20 |
20255.732813970742 ns/iter |
21280.30560862275 ns/iter |
0.95 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Catch2 Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
Fibonacci 10 |
145 ns (± 4) |
145 ns (± 1) |
1 |
Fibonacci 20 |
18.803 us (± 325) |
18.788 us (± 210) |
1.00 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
JMH Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
org.openjdk.jmh.samples.JMHSample_01_HelloWorld.wellHelloThere |
1971960426.4295356 ops/s |
1976042355.4545352 ops/s |
1.00 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Benchmark.Net Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
Sample.Benchmarks.Fib10 |
21.02941538508122 ns (± 0.11769825051195898) |
21.052492980773632 ns (± 0.023515474146354454) |
1.00 |
Sample.Benchmarks.Fib20 |
42.51694103678067 ns (± 0.22967208645508033) |
42.762034581257744 ns (± 0.07720966000547073) |
0.99 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Rust Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
bench_fib_10 |
175 ns/iter (± 1) |
175 ns/iter (± 1) |
1 |
bench_fib_20 |
22180 ns/iter (± 176) |
22175 ns/iter (± 3292) |
1.00 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Criterion.rs Benchmark
| Benchmark suite | Current: 49be1e5 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
BenchFib10 |
150 ns/iter (± 24) |
149 ns/iter (± 19) |
1.01 |
BenchFib20 |
18804 ns/iter (± 155) |
18804 ns/iter (± 2088) |
1 |
Fibonacci/Recursive/20 |
18818 ns/iter (± 1231) |
18809 ns/iter (± 345) |
1.00 |
Fibonacci/Iterative/20 |
3 ns/iter (± 0) |
4 ns/iter (± 0) |
0.75 |
Fibonacci/Recursive/21 |
30448 ns/iter (± 41) |
30435 ns/iter (± 57) |
1.00 |
Fibonacci/Iterative/21 |
4 ns/iter (± 0) |
4 ns/iter (± 0) |
1 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark 'Benchmark.js Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.
| Benchmark suite | Current: 362fe82 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
fib(10) |
13323 ops/sec (±0.41%) |
1614900 ops/sec (±0.33%) |
121.21 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark 'Benchmark.Net Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.
| Benchmark suite | Current: 362fe82 | Previous: a6c4f1c | Ratio |
|---|---|---|---|
Sample.Benchmarks.Fib10 |
42.24651443106787 ns (± 0.24835811309062725) |
21.052492980773632 ns (± 0.023515474146354454) |
2.01 |
This comment was automatically generated by workflow using github-action-benchmark.
ktrz
changed the title
closes #98