search

w1ld3r / crypto-signal

The #1 Automated Technical Analysis (TA) & Algo. Trading tool for Coinbase, Bittrex, Binance, and more! (Track over 500+ coins)
http://cryptosignal.dev
MIT License
78 stars 37 forks source link

Merge changes from rd-mg #62

Open w1ld3r opened 1 year ago

w1ld3r commented 1 year ago

Thanks for the new commits @rd-mg !

rd-mg commented 11 months ago

Not sorry, I did this push by mistake

On Fri, Nov 24, 2023, 2:28 PM w1ld3r @.***> wrote:

@.**** commented on this pull request.

Not pushable

On app/requirements-step-2.txt https://github.com/w1ld3r/crypto-signal/pull/62#discussion_r1404578249:

is there issue with higher jinja2 version ?

On app/analyzers/indicators/roc.py https://github.com/w1ld3r/crypto-signal/pull/62#discussion_r1404581344:

line 18: period_count (int, optional): Defaults to 15. Is there a bad period_count default value or a wrong value in the description ?

On app/config copy.bk2 https://github.com/w1ld3r/crypto-signal/pull/62#discussion_r1404582315:

Is it a wrong commit containning telegram key 1737113137:AA[...]OY ?

— Reply to this email directly, view it on GitHub https://github.com/w1ld3r/crypto-signal/pull/62#pullrequestreview-1748305330, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJMEYOXO6BCMLN6VQWZ5NF3YGDRLZAVCNFSM6AAAAAA3KT5TOWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBYGMYDKMZTGA . You are receiving this because you were mentioned.Message ID: @.***>