MMM-Fitbit2 PythonShell is not a constructor

frosty_snow

@Tippon
Hello - just wanted to inquire about this problem as I’m encountering the same thing but even with the mentioned change to the require statement it is continuing to occur.

Did you need to make any other changes or corrections to any of the files?
Thank you!

sdetweil

@frosty_snow can us do , from the module folder

grep version   node_modules/python_shell/package.json

frosty_snow

@sdetweil
Looks like I’m on 3.0.0

dmolony

@frosty_snow Not sure if helpful but I’m using python-shell 0.5.0 and removing the brackets around PythonShell fixed the same error for me.

Stoffbeuteluwe

var NodeHelper = require("node_helper");
const PythonShell = require("python-shell");

this is working for me

Vizulize

I have updated the module: https://github.com/m-roberts/MMM-Fitbit2/commit/27706ea0a9ab13b8d12a020f7234adf361d108ae

sdetweil

@vizulize i think it depends on what version of python-shell you are running…

https://github.com/extrabacon/python-shell/issues/154

your package.json says

"python-shell": "^0.5.0"

which means 0.5.0 OR ABOVE (technically ‘at minimum’)

if u want ONLY 0.5.0, take the ^ off the front…
else sometimes u will get 1.x

Vizulize

@sdetweil It is defined as 0.5.0 in the dependencies.

To be honest, the Python should be rewritten in JS anyway. Perhaps I will get around to doing that in the near future :thinking_face:

Then this issue goes away!

sdetweil

@vizulize yeh, i don’t know whay there is a python_shell lib… as exec and spawn are so easy…