Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix service_equal error reporting #486

Merged
merged 1 commit into from
Oct 26, 2023
Merged

fix service_equal error reporting #486

merged 1 commit into from
Oct 26, 2023

Conversation

chenyan-dfinity
Copy link
Contributor

assert_length should compare label, not the whole item.

@github-actions
Copy link

Benchmark for baecaf3

Click to view benchmark
Test Base PR %
Blob/&str 265.3±75.15µs 248.4±72.67µs -6.37%
Blob/ByteBuf 170.8±54.88µs 161.1±49.58µs -5.68%
Blob/Bytes 126.8±51.90µs 115.1±47.33µs -9.23%
Blob/String 343.6±168.24µs 323.4±159.95µs -5.88%
Collections/vec (text, nat) 82.4±1.58ms 80.7±2.90ms -2.06%
Collections/vec int 38.1±0.63ms 37.3±1.25ms -2.10%
Collections/vec int64 21.5±0.66ms 21.8±0.48ms +1.40%
Collections/vec nat8 17.0±0.23ms 16.9±0.27ms -0.59%
option list/1024 1548.0±34.62µs 1570.3±26.03µs +1.44%
profiles/1024 3.1±0.05ms 3.2±0.06ms +3.23%
variant list/1024 1256.5±20.12µs 1236.3±17.67µs -1.61%

@chenyan-dfinity chenyan-dfinity merged commit 782c437 into master Oct 26, 2023
4 checks passed
@chenyan-dfinity chenyan-dfinity deleted the fix-equal branch October 26, 2023 18:15
chenyan-dfinity added a commit that referenced this pull request Oct 27, 2023
* remove num_enum

* clean up parser dependencies

* checkpoint

* value feature

* remove codespan

* fix

* fix

* pretty

* remove mute_warning

* fix

* pretty refactor

* fix

* reexport candid in parser

* fix

* fix

* fix

* bignum feature

* bump candid_derive

* silence subtyping in decoder

* fix service_equal error reporting (#486)

* add IDLValue::Blob

* builder for config

* fix

* fix
chenyan-dfinity added a commit that referenced this pull request Oct 31, 2023
* fix service_equal error reporting (#486)

* Add Arbitrary for Principal and extend random for Func

* Use  instead of

* fmt

* add arbitrary for principal

---------

Co-authored-by: Yan Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants