Skip to content

Conversation

@zsfelfoldi
Copy link
Contributor

@zsfelfoldi zsfelfoldi commented Feb 19, 2021

This PR renames package lespay because that name did not sound good and was hard to say. Now I think we should separate three distinct things related to incentivization. There is the protocol (LES), the market mechanics and the payment channel. The economic model and market mechanics are not strictly tied to either the LES protocol or the payment channel technology. So I think the payment technology can be called LotteryBook (I think Gary gave it this name and I like it). For the market mechanics and the underlying model I had a few name ideas earlier, currently I think I'd like to call it "variflux" or "vflux", "vfx" for short. The reasoning behind it is that it models the network as a flow graph with constantly varying capacities. Also I think it sounds better and it's easy to abbreviate and the abbreviations sound ok too. The talk request ID of the UDP protocol could also be "vfx".
If you don't like it then I'm still somewhat open to suggestions, though this is the part that I primarily consider my own brainchild and I'd like to have a say in naming it :)

Note: "lespayDb" is not renamed in this PR because it is already renamed in another one.

les/client.go Outdated
Copy link
Member

Choose a reason for hiding this comment

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

vfc?

Copy link
Member

Choose a reason for hiding this comment

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

or vflux?

Copy link
Member

Choose a reason for hiding this comment

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

I think vflux perhaps is better? It's a common name in client side vfc and server side vfs

Copy link
Member

@rjl493456442 rjl493456442 left a comment

Choose a reason for hiding this comment

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

code wise, lgtm

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.

2 participants