Skip to content
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

2nd copy running with error #11

Closed
derrey opened this issue May 14, 2021 · 9 comments
Closed

2nd copy running with error #11

derrey opened this issue May 14, 2021 · 9 comments

Comments

@derrey
Copy link

derrey commented May 14, 2021

I have had to create a 2nd folder for a 2nd 3C account. When I run the same program in the 2nd folder I am getting this error.
I also tested it by copying my config file over to the new folder and didn't work, so I know the config is good.
Any thoughts?
I did do a new git copy....

get_bot_signals function took 2656.700 ms
'NoneType' object has no attribute 'lower'
Traceback (most recent call last):
File "run.py", line 538, in
ret = run_account(sub_account, bots)
File "/mnt/TradeBotJim/TradeBot/timeout.py", line 18, in wrapper
result = func(*args, **kwargs)
File "run.py", line 269, in run_account
if account_dict['signal_top_pairs'].lower() in ["min", "max"]:
AttributeError: 'NoneType' object has no attribute 'lower'

@derrey
Copy link
Author

derrey commented May 14, 2021

This must be a version update issue and maybe something needs to be added to the config file now? I copied the older version to the folder and it works fine. I will run the older version until I get confirmation on how I should proceed?

@JeeZues
Copy link
Owner

JeeZues commented May 14, 2021

Can you post your config file (without the api info obviously)? Are you running the latest code from the repo or the latest release when you saw the failure?

@derrey
Copy link
Author

derrey commented May 14, 2021

Can you post your config file (without the api info obviously)? Are you running the latest code from the repo or the latest release when you saw the failure?

I am using the git pull/fetch from the repo. then had to go back to my backup version before update.


'''


************ Copy this file rename_me_run_config.py to run_config.py before adding your info ***************


Create an API key/secret from https://www.binance.com/en/my/settings/api-management with only Enable Reading
one per account/sub
'''

Binance_APIs = [
{
'account_name': 'Jim Full - Futures1 USDT-M'
,'Binance_API_KEY': 'xxxxxx'
,'Binance_API_SECRET': 'xxxxxxx'
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs': True
,'no_short': True
}
]

'''
Create an API key/secret from https://3commas.io/api_access_tokens/new with BotsRead, BotsWrite, and AccountsRead
'''
TCommas_API_KEY = 'xxxxx'
TCommas_API_SECRET = 'xxxxxx'

'''
Optional
'''
ifttt_url = 'https://maker.ifttt.com/trigger/Binance_Futures_Alert/with/key/pslrXaf5UO6lNRH0jh1aoEhc8FZ_lX0x6xxxxxxxxxx'

@derrey
Copy link
Author

derrey commented May 14, 2021

Also, I was using this as the execution:

python3 run.py --show_deals --colors --auto --keep_running --keep_running_timer 15 --pair_allowance 500 --binance_account_flag ALL_SUBS --do_transfer --transfer_at 5000 --transfer_delta 100 --bot_start_bursts 10 --no_short

@JeeZues
Copy link
Owner

JeeZues commented May 14, 2021

Hi @derrey, The value of signal_top_pairs changed to be a string. It can be 'min', 'max', or anything that is not min or max, e.g. 'by_count'. See #10 for more info.

@derrey
Copy link
Author

derrey commented May 15, 2021

Sorry, I may have missed something. What is the best way to have the config file setup so some accounts have --signal_top_pairs, --signal_top_pairs_rnd, etc. Also, I keep getting the error and I am sure I am doing something simple wrong. :)

How should the config file look?
I have tried several different methods.. all have errors.

Config Info:

Binance_APIs = [
{
'account_name': 'Binance Dave_Sub02 Futures USDT-M'
,'Binance_API_KEY': 'xxxxx'
,'Binance_API_SECRET': 'xxxx'
# Optional argument name and value to override command line ones if multiple flags are used.
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs_rnd': 50
,'no_short': True
}
,{
'account_name': 'Binance Dave_Sub04 Futures USDT-M'
,'Binance_API_KEY': 'xxxxx'
,'Binance_API_SECRET': 'xxxxx'
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs': 'max'
,'no_short': True
}
,{
'account_name': 'Binance Dave_Sub05 Futures USDT-M'
,'Binance_API_KEY': 'xxxxx'
,'Binance_API_SECRET': 'xxxxxx'
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs': 'min'
,'no_short': True
}
,{
'account_name': 'Binance Dave_Sub06 Futures USDT-M'
,'Binance_API_KEY': 'xxxxxx'
,'Binance_API_SECRET': 'xxxxxxx'
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs': 'by_count'
,'no_short': True
}
,{
'account_name': 'Kelly Hu Futures1 USDT-M'
,'Binance_API_KEY': 'xxxxxxx'
,'Binance_API_SECRET': 'xxxxxxx'
,'stop_at': 2.0
,'start_at': 1.75
,'bots_per_position_ratio': 5
,'signal_top_pairs': 'by_count'
,'no_short': True
}
]

Error:

image

@JeeZues
Copy link
Owner

JeeZues commented May 15, 2021

@derrey, Try the latest code, should fix the issue. If it didn't you can specify the signal_top_pairs option in the config for now and I can look into it a bit more.
I need to add some testing code but who likes doing that :D

@derrey
Copy link
Author

derrey commented May 15, 2021

@derrey, Try the latest code, should fix the issue. If it didn't you can specify the signal_top_pairs option in the config for now and I can look into it a bit more.
I need to add some testing code but who likes doing that :D

That seemed to work.. Thanks.

@derrey
Copy link
Author

derrey commented May 15, 2021

@derrey, Try the latest code, should fix the issue. If it didn't you can specify the signal_top_pairs option in the config for now and I can look into it a bit more.
I need to add some testing code but who likes doing that :D

That seemed to work.. Thanks.

@JeeZues It is still working so I will close this issue.. Thanks

@derrey derrey closed this as completed May 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants