santiment / san-sdk

Code samples for working with Santiment's API and libraries
https://academy.santiment.net
MIT License
22 stars 13 forks source link

gap in MKR prices #20

Closed josefansinger closed 4 years ago

josefansinger commented 4 years ago

The metric daily_avg_price_usd for the slug maker is available since 2017-01-29. But you report no price data for the first months. Would it be better to report 2018-05-30 as availableSince when MKR did hit the exchanges?

IvanIvanoff commented 4 years ago

Our pricing data comes from coinmarketcap and there's gap in the maker prices there, too, that's where the gap comes from: image image

Maybe 21 November 2017 is a better availableSince, otherwise 7 months of data?

josefansinger commented 4 years ago

On 2017-11-21 your reported price is zero. Not what I can see at coinmarketcap. Then 2017-11-21 would be a good availableSince.

FYI, right in the middle on 2018-09-17 there is also a missing price point which is reported in coinmarketcap.

IvanIvanoff commented 4 years ago

I've scheduled a price refill for that time, let's wait some time to see if it's able to get the missing prices. Would take some time as there are thousands of projects and a very low rate limit to cmc.

IvanIvanoff commented 4 years ago

@josefansinger now I actually saw that you're using the daily_avg_price_usd metric instead of price_usd. The price is available, check this out.

The daily_avg_price_usd is something that should be deprecated from the API because of two reasons:

PS: There are still some gaps that exist in coinmarketcap but we have not data to fill them, so only the availableSince field change should be addressed

josefansinger commented 4 years ago

@IvanIvanoff Thank you.

I am now calling price_usd, volume_usd, daily_price_low_usd, daily_price_high_usd, daily_price_opening_usd and daily_price_closing_usd.

Any of them deprecated? Are historyPrice() and ohlc() deprecated?

josefansinger commented 4 years ago

If the availableSince field still needs to be addressed, shouldn't we reopen the issue?

IvanIvanoff commented 4 years ago

Let's create a separate issue as this one is gap in MKR prices.

I am now calling price_usd, volume_usd, daily_price_low_usd, daily_price_high_usd, daily_price_opening_usd and daily_price_closing_usd

With price_usd/volume_usd/marketcap_usd + aggregation you can fetch all of the daily_* metrics. For example:

high can be achieved with MAX aggregation and open can be achieved with FIRST aggregation.

All of the following are to be deprecated as a better version of them is available:

PS: https://github.com/santiment/san-sdk/issues/21 (Discussion only for availableSince)

josefansinger commented 4 years ago

Ah, ok. Opening = FIRST. Closing = LAST.

Then this issue can remain closed.

IvanIvanoff commented 4 years ago

You can explore all available aggregations per metric like this