Skip to content

Compute step and remaining duration from map.#150

Closed
afck wants to merge 1 commit into
vk-step-duration-transitionfrom
afck-step-duration-transition
Closed

Compute step and remaining duration from map.#150
afck wants to merge 1 commit into
vk-step-duration-transitionfrom
afck-step-duration-transition

Conversation

@afck
Copy link
Copy Markdown
Collaborator

@afck afck commented Jun 18, 2019

No description provided.


// Our current account nonce. The transactions must have consecutive nonces, starting with this one.
let mut tx_nonce = block.state.nonce(&our_addr)?;
let mut tx_nonce = block.state.nonce(&signer.address())?;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why do you think it is better to call address() twice instead of using the reference our_addr?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sorry, that was a mistake. I removed this part of the change.

@afck afck force-pushed the afck-step-duration-transition branch from 23e17a7 to 6148cfd Compare June 20, 2019 06:15
@afck afck marked this pull request as ready for review June 20, 2019 07:03
@afck
Copy link
Copy Markdown
Collaborator Author

afck commented Jun 20, 2019

The aura-pos-rebased version of this passes the posdao-test-setup tests for me, and changed the step duration from 5 to 3 seconds at 8:58 with:

"stepDuration": {
 "0": 5,
 "312202776": 3
},

@varasev
Copy link
Copy Markdown
Member

varasev commented Jun 24, 2019

Done in #152.

@varasev varasev closed this Jun 24, 2019
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