Skip to content

Updates toolchain to a more recent 2023-03-25 #4008

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Jun 2, 2023
Merged

Updates toolchain to a more recent 2023-03-25 #4008

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3c1d20c
Updates toolchain to a more recent 2023-03-25
Jun 1, 2023
2666794
Update faucet README
Jun 1, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions execution_engine_testing/tests/src/test/explorer/faucet.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -841,13 +841,13 @@ fn faucet_costs() {
// This test will fail if execution costs vary. The expected costs should not be updated
// without understanding why the cost has changed. If the costs do change, it should be
// reflected in the "Costs by Entry Point" section of the faucet crate's README.md.
const EXPECTED_FAUCET_INSTALL_COST: u64 = 71_691_403_720;
const EXPECTED_FAUCET_SET_VARIABLES_COST: u64 = 623_279_550;
const EXPECTED_FAUCET_CALL_BY_INSTALLER_COST: u64 = 3_101_326_690;
const EXPECTED_FAUCET_CALL_BY_USER_COST: u64 = 3_350_546_610;
const EXPECTED_FAUCET_INSTALL_COST: u64 = 75_226_865_930;
const EXPECTED_FAUCET_SET_VARIABLES_COST: u64 = 579_464_060;
const EXPECTED_FAUCET_CALL_BY_INSTALLER_COST: u64 = 3_040_141_320;
const EXPECTED_FAUCET_CALL_BY_USER_COST: u64 = 3_242_199_690;
Comment on lines +844 to +847
Copy link
Contributor

Choose a reason for hiding this comment

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

As per the comment above these consts, could you update the table in the indicated README too please?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done


let installer_account = AccountHash::new([1u8; 32]);
let user_account = AccountHash::new([2u8; 32]);
let user_account: AccountHash = AccountHash::new([2u8; 32]);

let mut builder = InMemoryWasmTestBuilder::default();
builder.run_genesis(&PRODUCTION_RUN_GENESIS_REQUEST);
Expand Down
2 changes: 1 addition & 1 deletion ...ution_engine_testing/tests/src/test/regression/host_function_metrics_size_and_gas_cost.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ const CONTRACT_TRANSFER_TO_ACCOUNT_U512: &str = "transfer_to_account_u512.wasm";
// This value is not systemic, as code is added the size of WASM will increase,
// you can change this value to reflect the increase in WASM size.
const HOST_FUNCTION_METRICS_STANDARD_SIZE: usize = 97_569;
const HOST_FUNCTION_METRICS_STANDARD_GAS_COST: u64 = 375_298_466_170;
const HOST_FUNCTION_METRICS_STANDARD_GAS_COST: u64 = 347_080_271_020;

/// Acceptable size regression/improvement in percentage.
const SIZE_MARGIN: usize = 5;
Expand Down
8 changes: 4 additions & 4 deletions smart_contracts/contracts/explorer/faucet/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ If you try to invoke the contract before these variables are set, then you'll ge

| feature | cost |
|---------|------------------|
| faucet install | `71_691_403_720` |
| faucet set variables | `623_279_550` |
| faucet call by installer | `3_101_326_690` |
| faucet call by user | `3_350_546_610` |
| faucet install | `75_226_865_930` |
| faucet set variables | `579_464_060` |
| faucet call by installer | `3_040_141_320` |
| faucet call by user | `3_242_199_690` |
2 changes: 1 addition & 1 deletion smart_contracts/rust-toolchain
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
nightly-2022-08-03
nightly-2023-03-25