-
Notifications
You must be signed in to change notification settings - Fork 206
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
docs(perp): Specification for client and concepts. #425
Conversation
…2) Add TODOs and polish other sections.
@@ -134,15 +134,15 @@ Returns: | |||
toll (sdk.Int): Amount of tokens transferred to the the fee pool. | |||
spread (sdk.Int): Amount of tokens transferred to the PerpEF. | |||
*/ | |||
func (k Keeper) CalcPerpTxFee(ctx sdk.Context, quoteAmt sdk.Dec) (toll sdk.Int, spread sdk.Int, err error) { | |||
func (k Keeper) CalcPerpTxFee(ctx sdk.Context, quoteAmt sdk.Dec) (toll sdk.Int, spread sdk.Int) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This error value was unused.
@@ -231,8 +231,6 @@ type VpoolKeeper interface { | |||
pair common.AssetPair, | |||
) (price sdk.Dec, err error) | |||
|
|||
CalcPerpTxFee(ctx sdk.Context, pair common.AssetPair, quoteAmt sdk.Int, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is not part of the VpoolKeeper. It's part of the PerpKeeper.
@@ -28,21 +28,6 @@ type Keeper struct { | |||
pricefeedKeeper types.PricefeedKeeper | |||
} | |||
|
|||
//CalcPerpTxFee calculates the total tx fee for exchanging 'quoteAmt' of tokens on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is unimplemented and unused.
Summary
Related