Webhook Notification AttributeError

[dwagenleiter]

Hey guys,

First of all, incredible job with this project. This takes away a lot of work while trying to do TA!

Unfortunately, while trying to send data / notifications to a webhook for further processing, I am getting the following error:

Traceback (most recent call last):
File "app.py", line 45, in
main()
File "app.py", line 39, in main
behaviour.run(settings['market_pairs'], settings['output_mode'])
File "/app/behaviour.py", line 64, in run
self.notifier.notify_all(new_result)
File "/app/notification.py", line 100, in notify_all
self.notify_webhook(new_analysis)
File "/app/notification.py", line 196, in notify_webhook
new_analysis[exchange][market][indicator_type][indicator][index] = analysis['result'].to_dict(orient='records')[-1]
AttributeError: 'dict' object has no attribute 'to_dict'

It is setup according to the documentation as:

NOTIFIERS_WEBHOOK_REQUIRED_URL=http://xxx/webhook.php

Any ideas what could cause this?

Best,
David

[ghost]

Sounds like a bug, are you able to provide what config options you are using so I can try to replicate it?

The webhook feature is quite new, so there's probably one or two nasties I haven't weeded out yet.

[dwagenleiter]

I am really just using the standard configuration described in the documentation, I didn't really get to that part yet as I wanted to write my logic first and test around.

SETTINGS_UPDATE_INTERVAL=300
SETTINGS_OUTPUT_MODE=json

SETTINGS_MARKET_PAIRS=ETH/BTC,BTC/USDT

NOTIFIERS_WEBHOOK_REQUIRED_URL=http://xxx/webhook.php

EXCHANGES_BITTREX_REQUIRED_ENABLED=true
INDICATOR_RSI_NUM_INDICATORS=2
INDICATOR_RSI_0_CANDLE_PERIOD=1d
INDICATOR_RSI_0_PERIOD_COUNT=14
INDICATOR_RSI_1_CANDLE_PERIOD=1h
INDICATOR_RSI_1_PERIOD_COUNT=14
INDICATOR_SMA_0_PERIOD_COUNT=9
INDICATOR_SMA_0_CANDLE_PERIOD=1h

[dwagenleiter]

Interestingly enough, putting NOTIFIERS_WEBHOOK_REQUIRED_URL=http://xxx/webhook.php to the end of the file fixed the issue.

[ghost]

That is quite weird... the config file is read in as environment variables so there really shouldn't be any reason for the order to matter... perhaps the issue is just inconsistent? But I will test both in either case and see if I can replicate :)

[dwagenleiter]

My bad, I temporarily changed SETTINGS_OUTPUT_MODE to CSV and that configuration didn't throw the error (my webhook did not get any data though).

I have changed it back to JSON now and I am getting the error again, each time.

[ghost]

Excellent, well I'll likely have it fixed within a week.

[dwagenleiter]

Awesome, thanks!

[ghost]

If you remove SETTINGS_OUTPUT_MODE=json from your config file completely it should work... some how the data set is being mutated when the output transform is being applied. I'm working on fixing that, but this should give you a work around in the mean time.

[ghost]

Fixed in #280

[ghost]

merged

[dwagenleiter]

Perfect, thank you!