Skip to content
This repository was archived by the owner on Aug 23, 2020. It is now read-only.

tip selection: set default alpha value to 0.001#817

Merged
GalRogozinski merged 1 commit intoiotaledger-archive:devfrom
alon-e:tune/TipSelectionAlphaValue
Jun 18, 2018
Merged

tip selection: set default alpha value to 0.001#817
GalRogozinski merged 1 commit intoiotaledger-archive:devfrom
alon-e:tune/TipSelectionAlphaValue

Conversation

@alon-e
Copy link
Copy Markdown
Contributor

@alon-e alon-e commented Jun 18, 2018

Description

This PR sets the default alpha value for the random walk to 0.001,
in order for slower but honest users to still have a good chance of getting their transactions confirmed.

This value is based on both:

  1. an analytical calculation of the probability of a transaction with slower PoW time H ~= 5mins to still get confirmed (assuming a constant lambda = 10 TPS and H >> h).

  2. validation (of calculation) by running a private testnet (w/ 12TPS spam) where slower transactions were issued and confirmation rates were measured.

Fixes #801

Type of change

  • Parameter tuning

How Has This Been Tested?

see Description

Checklist:

Please delete items that are not relevant.

  • My code follows the style guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@alon-e alon-e added C-Tip Selection Old C-Configuration Component - Configuration labels Jun 18, 2018
@alon-e alon-e added this to the Barcelona milestone Jun 18, 2018
@alon-e alon-e requested a review from GalRogozinski June 18, 2018 11:50
@GalRogozinski GalRogozinski merged commit 50ab035 into iotaledger-archive:dev Jun 18, 2018
GalRogozinski pushed a commit to GalRogozinski/iri that referenced this pull request Jun 21, 2018
GalRogozinski pushed a commit to GalRogozinski/iri that referenced this pull request Jun 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Validate the alpha value

2 participants