search

CryptoSignal / Crypto-Signal

Github.com/CryptoSignal - Trading & Technical Analysis Bot - 4,100+ stars, 1,100+ forks
https://www.linkedin.com/in/AbenezerMamo
MIT License
4.84k stars 1.26k forks source link

Error while doing your 5th step : Build your own Docker image #407

Closed bluetyphoon77 closed 3 years ago

bluetyphoon77 commented 3 years ago

I did this line like you recommend : docker build -t dev/crypto-signals:latest

And I have this error "docker build" requires exactly 1 argument. See 'docker build --help'. Usage: docker build [OPTIONS] PATH | URL | - Build an image from a Dockerfile

By the way, from which folder should I run this command ?

Thanks

laliux commented 3 years ago

Hey @bluetyphoon77

I think you are missing a DOT at the end of docker command.

docker build -t dev/crypto-signals:latest .

Run from root folder of the proyect.

Best

bluetyphoon77 commented 3 years ago

Thanks a lot Eduardo ! It works now :) But do you know why it uses my coins BUT send me ALL notifs on ALL indicators at intervals I did not set,... ? LOL

Thanks alot

From: Eduardo Zarate notifications@github.com Sent: Sunday, November 29, 2020 10:34 PM To: CryptoSignal/Crypto-Signal Crypto-Signal@noreply.github.com Cc: bluetyphoon77 magicyvan@hotmail.com; Mention mention@noreply.github.com Subject: Re: [CryptoSignal/Crypto-Signal] Error while doing your 5th step : Build your own Docker image (#407)

Hey @bluetyphoon77https://github.com/bluetyphoon77

I think you are missing a DOT at the end of docker command.

docker build -t dev/crypto-signals:latest .

Run from root folder of the proyect.

Best

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/CryptoSignal/Crypto-Signal/issues/407#issuecomment-735468587, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEEGYFSHORQWDJFUFO322LSSLD6FANCNFSM4UGRPHOA.

laliux commented 3 years ago

@bluetyphoon77

I need to see your config file so I can try to identify the problem. In the mean time check this sample config file: https://pastebin.com/xEa5m2sZ

Best

bluetyphoon77 commented 3 years ago

Thanks again for your answer. My config is so simple compared to yours 😉 But where should I put it ? in app folder ? or main cryptosignal folder ?

here it is attached

I left THETABTC to test But I still have those notifs LOL [cid:36604a04-479d-48e2-b6c5-f3a3a58e77d5]

From: Eduardo Zarate notifications@github.com Sent: Monday, November 30, 2020 3:03 AM To: CryptoSignal/Crypto-Signal Crypto-Signal@noreply.github.com Cc: bluetyphoon77 magicyvan@hotmail.com; Mention mention@noreply.github.com Subject: Re: [CryptoSignal/Crypto-Signal] Error while doing your 5th step : Build your own Docker image (#407)

@bluetyphoon77https://github.com/bluetyphoon77

I need to see your config file so I can try to identify the problem. In the mean time check this sample config file: https://pastebin.com/xEa5m2sZ

Best

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/CryptoSignal/Crypto-Signal/issues/407#issuecomment-735517981, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEEGYE5JQFBS3BEPLPG62LSSMDRRANCNFSM4UGRPHOA.

laliux commented 3 years ago

Config file must be in "app" folder.

Is not necesary to rebuild after each change on config file, just stop your running docker and run again.

You should explicitly disable all indicators you dont use. Check app/defaults.yml as reference.

Increase your update_interval: 300, this time is in seconds. With that value you are running analysis each 5mins, but you are using 1d candle periods,. doesn't make much sense.

Best

bluetyphoon77 commented 3 years ago

Thanks again Eduardo ! I was doing more tests ... and you confirm what I realized about the disable ! ;) thanks I succeeded to get the right signals. But when I tri the conditionals, I still have each separate signal and not the combination signal... would be great to have your advice on this last step for me ;)

I checked here : https://github.com/w1ld3r/crypto-signal/commit/069cc314466bb6ddba78dc36c41aa1e7111f222c But did not help

thanks again PS: you are right for 300 ;) now I4m using the test environment ;)

laliux commented 3 years ago

Conditionals are not available in this project, hopefully soon I can integrate changes by @w1ld3r.

bluetyphoon77 commented 3 years ago

But it is already integrated in the fork ! Link is above ;) I just don't find how to contact dev to know how to make it work :(

w1ld3r commented 3 years ago

Hello @bluetyphoon77, for any problem using my fork please create an issue in the fork. Also before creating an issue please take a look at the documentation -> https://github.com/w1ld3r/crypto-signal/blob/develop/docs/config.md#8-conditionals PS: You can now create an issue on my fork (I had not activated the feature ...)

bluetyphoon77 commented 3 years ago

Hi Eduardo !!! I've managed to understand and find all solutions besides ONE ! I'd love if you can help. I see you use STOCHRSI in your config. Why are you using it ? Because I've tested it and getting only wrong values, for stoch, k, and d ! Have you checked the values and compared them with tradingview ? Cheeers

From: Eduardo Zarate notifications@github.com Sent: Monday, November 30, 2020 3:03 AM To: CryptoSignal/Crypto-Signal Crypto-Signal@noreply.github.com Cc: bluetyphoon77 magicyvan@hotmail.com; Mention mention@noreply.github.com Subject: Re: [CryptoSignal/Crypto-Signal] Error while doing your 5th step : Build your own Docker image (#407)

@bluetyphoon77https://github.com/bluetyphoon77

I need to see your config file so I can try to identify the problem. In the mean time check this sample config file: https://pastebin.com/xEa5m2sZ

Best

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/CryptoSignal/Crypto-Signal/issues/407#issuecomment-735517981, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEEGYE5JQFBS3BEPLPG62LSSMDRRANCNFSM4UGRPHOA.

laliux commented 3 years ago

The file I shared with you is just an example, it is not my own configuration file.

As I already mentioned in another thread, I will review the StochRSI when I have a little time :)

bluetyphoon77 commented 3 years ago

Thanks for your answer Eduardo. I also sent an issue into the TA-lib support. It seems they have problems with STOCHRSI. But I don't understand their programmer language LOL, so it will be useful if you can translate to me the solution, even if it means not using this TA-lib anymore and chosing another library ! Thansk a lot 😉

From: Eduardo Zarate notifications@github.com Sent: Thursday, December 3, 2020 4:20 AM To: CryptoSignal/Crypto-Signal Crypto-Signal@noreply.github.com Cc: bluetyphoon77 magicyvan@hotmail.com; Mention mention@noreply.github.com Subject: Re: [CryptoSignal/Crypto-Signal] Error while doing your 5th step : Build your own Docker image (#407)

The file I shared with you is just an example, it is not my own configuration file.

As I already mentioned in another thread, I will review the StochRSI when I have a little time :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/CryptoSignal/Crypto-Signal/issues/407#issuecomment-737655717, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEEGYBPNCKQUTJ6GMJ4X4TSS4GXNANCNFSM4UGRPHOA.