Skip to content
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.

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

Adjust cost per signature #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Archimedes1618
Copy link
Collaborator

Lowered cost per signature

from

Safecoin Target: 10 x 200.000 = 2.000.000 Lamports per Slot (min 1.000.000 - max 20.000.000)

to

Safecoin Target: 10 x 50.000 = 500.000 Lamports per Slot (min 250.000 - max 5.000.000)

This is an adjustment in correlation with PR #8

Lowered cost per signature

 from

 Safecoin Target: 10 x 200.000 = 2.000.000 Lamports per Slot (min 1.000.000 - max 5.000.000)

to

 Safecoin Target: 10 x 50.000 = 500.000 Lamports per Slot (min 250.000 - max 5.000.000)

This is an adjustment in correlation with PR #8
@Archimedes1618 Archimedes1618 self-assigned this May 29, 2021
Slot distance increased by 50% to complement lowered hardware requirements
@araviel-io
Copy link

I'm in, very good idea if that's reducing delinquency rate and stability. Since Solana have more than 8* our supply, modfying fees accordingly is smart !

@Fair-Exchange
Copy link
Collaborator

Still thinking on this one, there are a number of factors at play including burn rate.

at 200_000 lamports per signature I believe we need something like 500 tps to acheive steady state supply (burn cancels out minting), I need to double-check those numbers. Ultimately I am wondering if Solana actually set their fees too low, maybe we can discuss this more.

I'm getting ready for our move to v1.6.16 and reviewing all PRs. I'd like to hold onto this one a bit longer if possible

@Danielale1
Copy link

Ready

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.

None yet

4 participants