Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

Swap phase 1 #611

Closed
gbalint opened this issue May 25, 2018 · 6 comments · Fixed by ethereum/go-ethereum#18337
Closed

Swap phase 1 #611

gbalint opened this issue May 25, 2018 · 6 comments · Fixed by ethereum/go-ethereum#18337
Milestone

Comments

@gbalint
Copy link

gbalint commented May 25, 2018

Swap, Swear and Swindle is the incentivization system for Swarm. For details, see https://swarm-gateways.net/bzz:/theswarm.eth/ethersphere/orange-papers/1/sw%5E3.pdf

In a nutshell, swap is an accounting protocol to keep track of the downloads between two peers. We use the number of chunks requested and retrieved as a discrete integer
unit of accounting. In the ideal scenario of compliant use, the balance is kept around zero. When the mutual balance on a given connection is tilted in favour of one peer, that peer should be compensated in order to bring the balance back to zero.

Swap phase 1 is about keeping track of this balance

@gbalint gbalint changed the title Swap Swap phase 1 May 25, 2018
@gbalint gbalint added swap and removed sw3 labels May 25, 2018
@cobordism cobordism added this to the 0.3.1 milestone Jun 7, 2018
@gbalint gbalint removed this from the 0.3.1 milestone Aug 2, 2018
@holisticode
Copy link
Contributor

Swap phase 1 is getting accounting to work.

@kkemenczy kkemenczy added this to the 0.4 milestone Aug 8, 2018
@gbalint gbalint modified the milestones: 0.4, 0.3.3 Aug 8, 2018
@kkemenczy kkemenczy modified the milestones: 0.4, 0.4.1 Aug 8, 2018
@acud
Copy link
Member

acud commented Nov 25, 2018

@holisticode isn't this already done? can we close?

@nonsense
Copy link
Contributor

@holisticode maybe we should send the accounting registry to InfluxDB as well (optionally), so that we can visualise the balances. At the moment they are persisted in a file, but we don't really have any intuition about the numbers. What do you think?

@holisticode
Copy link
Contributor

It is not finished yet. The last step to close this ticket (in terms of functionality) is #1026.

This PR introduces the actual balance tracking and pricing for the stream protocol.

But that PR includes the first real cryptoeconomic element of swap - the price of a message. It currently contains a arbitrary price for a message which has no base in cryptoeconomics and should be discussed. We should do this ASAP @zelig @nagydani @homotopycolimit

@holisticode
Copy link
Contributor

@nonsense I agree it may be interesting to do. We would need to consider the performance and conceptual overhead of doing that. I don't think the performance overhead is huge, but I guess we should get buy-in for such a proposal.

@nonsense
Copy link
Contributor

In my opinion there is no performance overhead at all - we are just checking 10 integers every 5 or 10sec.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants