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

refactor get_price #1276

Merged
merged 3 commits into from
Jul 30, 2021
Merged

refactor get_price #1276

merged 3 commits into from
Jul 30, 2021

Conversation

wangjj9219
Copy link
Member

close #1274

@codecov
Copy link

codecov bot commented Jul 29, 2021

Codecov Report

Merging #1276 (dbecbbc) into master (08042f6) will decrease coverage by 0.45%.
The diff coverage is 71.48%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1276      +/-   ##
==========================================
- Coverage   66.82%   66.37%   -0.46%     
==========================================
  Files         225      222       -3     
  Lines       20095    19801     -294     
==========================================
- Hits        13428    13142     -286     
+ Misses       6667     6659       -8     
Impacted Files Coverage Δ
modules/cdp-engine/src/lib.rs 53.33% <ø> (ø)
modules/cdp-engine/src/mock.rs 76.47% <ø> (+2.88%) ⬆️
modules/honzon/src/mock.rs 58.97% <ø> (+2.87%) ⬆️
modules/nft/src/benchmarking.rs 0.00% <ø> (ø)
modules/nft/src/mock.rs 37.03% <ø> (ø)
modules/nft/src/weights.rs 0.00% <0.00%> (ø)
modules/transaction-payment/src/weights.rs 0.00% <0.00%> (ø)
runtime/acala/src/lib.rs 6.20% <0.00%> (ø)
runtime/acala/src/weights/module_nft.rs 0.00% <0.00%> (ø)
...me/acala/src/weights/module_transaction_payment.rs 0.00% <0.00%> (ø)
... and 52 more

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 bc8a777...dbecbbc. Read the comment docs.

@xlc
Copy link
Member

xlc commented Jul 29, 2021

This is a bit confusing. Maybe we can just have two price provider?

@wangjj9219 wangjj9219 merged commit cb02ba7 into master Jul 30, 2021
@wangjj9219 wangjj9219 deleted the refactor-prices branch July 30, 2021 16:08
syan095 pushed a commit that referenced this pull request Aug 10, 2021
…tegration-test

* origin/master: (27 commits)
  Feature/homa lite improvement (#1308)
  payout no longer transfer rewards to owner directly (#1312)
  Tx priority order (#1304)
  to check the required collateral ratio only when withdraw collateral or issue more debit (#1311)
  fix interest calculation
  Improve ProxyType (#1288)
  set max collateral when close cdp has debit by dex (#1296)
  use the slippage for the price from oracle to limit internal swap (#1285)
  Update serde rule of some enum which is the param type of rpc to camel case (#1291)
  set fee swap path should prevent starting from native token (#1293)
  remove migration impl of transaction-payment (#1287)
  remove migration of session-manager and enable it (#1289)
  set end_time when create collateral auction (#1282)
  Update mandala (#1280)
  Feature/homa lite improvement (#1239)
  Remove KAR airdrop. (#1266)
  update parameters (#1279)
  refactor get_price (#1276)
  Fix coverage CI (#1267)
  reduce dex liquidation slippage (#1273)
  ...
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.

Refactor the interface of get price, can choose to priority to get locked price or get real-time price
2 participants