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

les: rename lespay to vflux #22347

Merged
merged 1 commit into from
Feb 19, 2021
Merged

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
@@ -263,9 +263,9 @@ func (s *LightEthereum) APIs() []rpc.API {
Service: NewPrivateLightAPI(&s.lesCommons),
Public: false,
}, {
Namespace: "lespay",
Namespace: "vfx",
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