Skip to content

Conversation

@JamesHinshelwood
Copy link
Contributor

Add Scilla map edge case test from Chetan. It currently fails.

Add Scilla map edge case test from Chetan. It currently fails.
.request("GetSmartContractState", [contract_address])
.await
.unwrap();
info!("State: {state}");
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you want to assert on sth here perhaps?

.request("GetSmartContractState", [contract_address])
.await
.unwrap();
info!("State: {state}");
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used these logs when debugging the state problems to find out what the state was at each stage. The failures usually happen in the send_transaction call when the transaction is not successful.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

🐰 Bencher Report

Branchscilla-map-test
Testbedself-hosted
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
full-blocks-erc20-transfers/full-blocks-erc20-transfers📈 view plot
🚷 view threshold
1,045.20 ms
(-17.55%)Baseline: 1,267.62 ms
1,665.98 ms
(62.74%)
full-blocks-evm-transfers/full-blocks-evm-transfers📈 view plot
🚷 view threshold
469.04 ms
(-11.14%)Baseline: 527.81 ms
668.24 ms
(70.19%)
full-blocks-giant-deploy/full-blocks-giant-deploy📈 view plot
🚷 view threshold
6,226.60 ms
(-13.04%)Baseline: 7,160.54 ms
9,369.89 ms
(66.45%)
full-blocks-scilla-add/full-blocks-scilla-add📈 view plot
🚷 view threshold
1,081.20 ms
(-12.91%)Baseline: 1,241.47 ms
1,638.55 ms
(65.99%)
full-blocks-scilla-load/full-blocks-scilla-load📈 view plot
🚷 view threshold
3,857.30 ms
(-8.12%)Baseline: 4,198.24 ms
5,178.99 ms
(74.48%)
full-blocks-scilla-store/full-blocks-scilla-store📈 view plot
🚷 view threshold
3,300.00 ms
(-6.30%)Baseline: 3,521.96 ms
4,245.17 ms
(77.74%)
full-blocks-zil-transfers/full-blocks-zil-transfers📈 view plot
🚷 view threshold
2,261.30 ms
(-24.27%)Baseline: 2,986.09 ms
5,473.58 ms
(41.31%)
process-empty/process-empty📈 view plot
🚷 view threshold
11.58 ms
(-6.03%)Baseline: 12.32 ms
13.87 ms
(83.48%)
🐰 View full continuous benchmarking report in Bencher

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.

3 participants