-
Notifications
You must be signed in to change notification settings - Fork 45
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
[Hero & Hertz] 'bitshares-pricefeed' referencing bitUSD | BTS_Tools referencing BTC:USD (bitUSD|BTC:USD) #33
Comments
From what I understand from the code, @xeroc's script use bitUSD market price. There is usually a 10% spread between the market price and the settlement price. Another approach could be to use bitUSD settlement price/current feed (= the median price of all the feeds). I think if you already publish the price of USD:BTS it's better to use your computed price (this way USD and HERTZ price will be consistent), but if you publish only HERTZ, the best it's to get bitUSD settlement price. I have added BitsharesFeed source for this purpose in my repo. |
BSIP 76 (bitshares/bsips#221) has now set fixed (fake) price feeds for bitUSD. Since this price feed script uses bitUSD as the price feed reference it's now producing inaccurate Hertz price feeds. Can an alternative to bitUSD be provided for Hertz to become accurate again? |
Note that using my fork (which is used by most of the witnesses), the USD price used depends of the configuration file.
|
Summarized snippets of Wackou's price feed script:
Current Hero&Hertz Yaml USD reference:
There's a difference in price BTS:USD reference between the two Bitshares price feed libraries. @xeroc's scripts reference bitUSD (decentralized, reliant on bitUSD functioning properly, less volatile?) meanwhile @wackou's price feed scripts reference external BTC:USD price feeds (Reliant on centralized exchanges, potentially vulnerable to manipulation, potentially avoid following bitUSD into a blackswan).
What's the appropriate USD reference for Hero (and consequentially Hertz) bitUSD:BTS and/or BTC:USD?
Cheers
The text was updated successfully, but these errors were encountered: