search

bitreport-org / Bitreport

Automated technical analysis reports for cryptocurrencies 🚀
MIT License
8 stars 1 forks source link

Missing indicators #105

Closed glowacki-dev closed 5 years ago

glowacki-dev commented 5 years ago

https://sentry.io/organizations/bitreport-0x/issues/952071480/

Jak to powinno zostać rozwiązane? Prosze o RSI i nic nie dostaję (nawet klucza z RSI nie ma chyba)

turbaszek commented 5 years ago

Tak, musimy pomyśleć bo to wynika z tego, że przy braku danych zwracam samą cenę i volume :<

turbaszek commented 5 years ago

Btw. nie mamy timeframe 4h. W ogóle można by sie zastanowić czy bot w przypadku błędów nie powinien jakoś sie wytłumaczyć w stylu "Oh sorry I fucked up" 😄

glowacki-dev commented 5 years ago

https://github.com/nuclearpinguin/Bitreport/blob/master/dashboard/app/services/tweets/responder.rb#L30

turbaszek commented 5 years ago

Wydaje mi się, że #110 rozwiązało ten problem. Charting jak się nie znajdzie to zwróci puste słowniki ale one będą. Jeśli Talib się nie uda to wskaźnika nie będzie.

Głównie problem wynikał z tego, że na siłę próbowałem zapewnić, że jak chcesz 200 świec to masz dostać 200 świec, nie mniej. Teraz jak jest mniej to dostaniesz mniej.

Wciąż jeśli będzie mniej niż 15 świec to np. RSI się nie ogarnie, a z tego co mówiłeś jest ono potrzebne, żeby coś na dole rysować?

glowacki-dev commented 5 years ago

Sytuację typu "mniej niż 15 świec" to możemy pominąć na razie 🤷‍♀ Generalnie jeśli będę dostawał przynajmniej pusty obiekt pod każdym ze spodziewanych kluczy to będzie ok. Czyli RSI może nie być, ale wtedy w zwrotce spodziewam się czegoś typu:

{
  "RSI": {
    "rsi": []
  }
}
glowacki-dev commented 5 years ago

Analogicznie przy wszystkich innych wskaźnikach. Niech zawsze będą wszystkie klucze, a ewntualnie może nie być żadnych danych w środku

turbaszek commented 5 years ago

Wygląda na to, że działa :D Wszystkie wskaźniki puste: preview-5

glowacki-dev commented 5 years ago

O dokładnie tak powinno być 🚀