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

Removal of the *charge* portion of the economic model #365

Closed
Tracked by #1690
ureeves opened this issue Jun 10, 2024 · 0 comments
Closed
Tracked by #1690

Removal of the *charge* portion of the economic model #365

ureeves opened this issue Jun 10, 2024 · 0 comments

Comments

@ureeves
Copy link
Member

ureeves commented Jun 10, 2024

Summary

In the streamlining of the economic protocol described in dusk-network/rusk#1644, we've decided to do away with the charging portion, and as such we need to remove such functionality here as well.

Detailed Description (optional)

By merging #363 we introduced a charging component, recognized by the ABI, and formalized into an enum:

#[derive(Debug, Clone, Archive, Deserialize, Serialize, Eq, PartialEq)]
#[archive_attr(derive(CheckBytes))]
pub enum EconomicMode {
Allowance(u64),
Charge(u64),
None,
Unknown,
}

We would like to remove the Charge variant, and any code relevant to it. This will mean slight changes in the ABI, particularly in which modes are recognized. The set_charge function should also be removed.

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

No branches or pull requests

1 participant