rooch-network / rooch

VApp Container with Move Language
https://rooch.network
Apache License 2.0
128 stars 54 forks source link

ci(bench): refine bcs serialized size bench by passing function as put values #1568

Closed popcnt1 closed 1 month ago

popcnt1 commented 1 month ago

Summary

This commit refines the benchmarks for 'bcs serialized size' by making the benchmark function take function values as inputs. This is done by redefining the benchmark function to take a vector of function containers as parameter, which allows different evaluation functions to be passed in as arguments. It provides a more flexible way to perform benchmarks and measure different aspects of the encoding and serializing processes:

bcs_serialized_size/serialized_size
                        time:   [5.4677 ns 5.4749 ns 5.4862 ns]
                        change: [-0.9429% -0.4567% +0.0422%] (p = 0.11 > 0.05)
                        No change in performance detected.
Found 2 outliers among 10 measurements (20.00%)
  2 (20.00%) high mild
bcs_serialized_size/encode_size
                        time:   [33.733 ns 33.749 ns 33.763 ns]
                        change: [-1.1451% -1.0485% -0.9064%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 10 measurements (10.00%)
  1 (10.00%) high severe
vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **rooch** | ⬜️ Ignored ([Inspect](https://vercel.com/rooch/rooch/AZxwopjMhhptGdoRG9CcPYXTYut6)) | [Visit Preview](https://rooch-git-fork-popcnt1-cibench-rooch.vercel.app) | | Apr 13, 2024 8:04pm |