fix: tenderly node simulation block with (block number - 1)

[jsy1218]

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Bug fix

• What is the current behavior? (You can also link to an open issue here) From tenderly:

Yes, on the API, if you specify bn 100, we take the state of bn 99 and create a new block 100 where we simulate at the beginning. On the Node, if you specify bn 100, we take the state of block 100, and create a fake block 101 where we simulate.

• What is the new behavior (if this is a feature change)? Make (blocknumber - 1) for tenderly node simulation. From tenderly:

So I'd recommend adding the following changes to check if the diffs remain:

• add the block_number parameter to all sims in the API bundle
• use block_number - 1 for the Node bundle

• Other information:

[jsy1218]

• #629 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jsy1218 and the rest of your teammates on Graphite

[graphite-app[bot]]

Graphite Automations

"Request reviewers once CI passes on smart-order-router repo" took an action on this PR • (07/01/24)

4 reviewers were added and 1 assignee was added to this PR based on 's automation.