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

chore: update weights and reference weights #342

Merged
merged 5 commits into from
Dec 15, 2021

Conversation

enthusiastmartin
Copy link
Contributor

Weights updated for xyk,exchange and multi payment pallet together with reference weights used in check perf script.

"buy_extrinsic": 120.0,
"on_finalize_for_one_buy_extrinsic": 179.0
"sell_intention": 102.0,
"buy_intention": 142.0,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is really buy that much more expensive?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope. mistake. it is fixed. it is the same as sell.

@codecov
Copy link

codecov bot commented Dec 15, 2021

Codecov Report

Merging #342 (10666b1) into master (3a544b5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #342   +/-   ##
=======================================
  Coverage   15.31%   15.31%           
=======================================
  Files          20       20           
  Lines        6812     6812           
=======================================
  Hits         1043     1043           
  Misses       5769     5769           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 56bd7de...10666b1. Read the comment docs.

@enthusiastmartin enthusiastmartin merged commit 9d0af3e into master Dec 15, 2021
@enthusiastmartin enthusiastmartin deleted the chore/update-weigts branch December 15, 2021 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants