MMM-Temperature uncaught exception...

wishmaster270

your python script looks good to me and also the output of node_helper till the moment it tries to send the data to the frontend.
Can you provide your MagicMirror and nodejs version information to me, please?

wishmaster270

@Brown19

I tried your script and the current version of the module in both of my setups (2.24.0 native and 2.25.0-develop container) and do not have any problems.

As i wrote already the problem looks like there is problem during communication between the node_helper.js and the frontend (either electron or the browser of your choice).

Brown19

@wishmaster270 I’m running 2.24.0 and node is v18.17.0

wishmaster270

@Brown19 Do you see any errors in the development console of the browser?

wishmaster270

@Brown19
Hi,

today i installed a new instance of MagicMirror 2.24.0 with both your node version 18 any my current used one (16).
In the installation which uses electron i can reproduce the error now. But i am pretty sure this is a problem related to the combination of the version 2.24.0 and the used electron version.
I found this thread with the same error but MMM-Tools instead in the forum of bogusnet, too.
It might be a general MagicMirror issue and not a problem of the module.

Maybe one of the core developers can have a look into it. I never had been that deep in the MagicMirror code and do not have much time to investigate the problem at the moment.

sdetweil

@wishmaster270 what module is this? I’ll look at it tonight.

wishmaster270

@sdetweil Hi Sam, it looks like both MMM-Temperature and MMM-Tools cause the error.

wishmaster270

@sdetweil @Brown19
Hi,

i did some more investigation today and think i found a solution.
Although the exception looks different i think the problem is the execSync command.
The only way i was able to fix it is to use spawnSync.

But i can not say where or why the problem happens.
I tried different node versions (latest 16, 18, 20) and different electron versions (25.3.0, 25.3.2 and 25.4.0) with execSync.

I will release a new version of MMM-Temperature today which uses spawnSync instead of execSync.

sdetweil

@wishmaster270 good catch. I think exec is foreground, spawn is background.

wishmaster270

@Brown19

Just released the new version 0.0.8 which (hopefully) fixes the problem.

I integrated your script, too. But i named it sht31d.