Skip to content

Conversation

fakedev9999
Copy link

@fakedev9999 fakedev9999 commented Sep 11, 2025

This is a consensus breaking change.

Description

Updates operator fee formula to be more useful.

Tests

Updated TestNewOperatorCostFunc and TestTotalRollupCostFunc.

Additional context

Metadata

@fakedev9999 fakedev9999 requested a review from a team as a code owner September 11, 2025 07:14
@geoknee geoknee added the H-jovian Feature slated for Jovian hardfork label Sep 11, 2025
Copy link
Contributor

@geoknee geoknee left a comment

Choose a reason for hiding this comment

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

Needs some tests, too.

Copy link
Contributor

@geoknee geoknee left a comment

Choose a reason for hiding this comment

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

Looking good, just a couple of nits to resolve!

@sebastianst sebastianst changed the title fix!: multiply operatorFeeScalar by 100 instead of dividing by 1e6 core/types: Implement Jovian Operator Fee Fix Sep 17, 2025
@fakedev9999 fakedev9999 force-pushed the taehoon/update-operator-fee-formula branch from a15a068 to a75900f Compare September 18, 2025 23:42
@fakedev9999 fakedev9999 force-pushed the taehoon/update-operator-fee-formula branch from c291a65 to befc452 Compare September 25, 2025 03:08
@tynes
Copy link
Contributor

tynes commented Sep 25, 2025

This logic looks correct to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
H-jovian Feature slated for Jovian hardfork
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Scalar Fix] Implement specification
3 participants