Module Developer Challenge - I surrender!

Mykle1

@ninjabreadman said in Module Developer Challenge - I surrender!:

acpi seems like a contender, as does lm-sensors.

Made some progress with lm-sensors which gives me great hope. I can see the return in the terminal but not in the node_helper. Still, I’m encouraged. I’ll continue to work on it tomorrow. I’m feeling a bit tired tonight.

Sockets connected & modules started ...
Launching application.
{ 'nouveau-pci-0100': { 'PCI adapter': { temp1: [Object] } },
  'coretemp-isa-0000': 
   { 'ISA adapter': 
      { 'Core 0': [Object],
        'Core 1': [Object],
        'Core 2': [Object],
        'Core 3': [Object] } } }
Whoops! There was an uncaught exception...
TypeError: this.sendSocketNotification is not a function
    at /home/pop/MagicMirror/modules/MMM-PC-Stats/node_helper.js:32:9
    at /home/pop/MagicMirror/modules/MMM-PC-Stats/node_modules/sensors.js/lm-sensors.js:96:12
    at ChildProcess.exithandler (child_process.js:209:7)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:194:7)
    at maybeClose (internal/child_process.js:899:16)
    at Socket. (internal/child_process.js:342:11)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at Pipe._handle.close [as _onclose] (net.js:510:12)

E3V3A

@Mykle1 Not sure what you’re doing in your code. But…

You’ll get a temp reading for each cpu core, so you need to choose|add|average…, and
How do you execute the script? Did you give it proper permissions?
I’m you already know, but you also need to allow the external file to be read from node_helper.js.

Mykle1

@E3V3A said in Module Developer Challenge - I surrender!:

Not sure what you’re doing in your code. But…

Neither am I. :-)

strawberry 3.141

@Mykle1 you might want to upload your current version to a github branch so we can see what your doing

cowboysdude

He can’t it’s a secret HAHHAHAHAHHA

Mykle1

I would like to thank everyone that gave me suggestions and advice. I really do appreciate the support.

I ended up using a combination of 2 npm packages. pc-stats and sensors.js. My first time building a module from an npm package, and then adding a second to the same module confused me quite a bit. That, and my lack of any real coding experience, found me at a standstill.

Well, once again, @cowboysdude took pity on me. He took the time to teach me where I went wrong, and what I was missing, and what I didn’t need. Maybe I won’t retain everything that he taught me but I will certainly remember his kindness and generosity. His patience is astonishing. :-)

https://github.com/mykle1/MMM-PC-Stats

Now I just need some formatting if anyone cares to assist me with that. Anyway, here’s a quick shot. Ideally, I would like to move the Core temps to right, just under the video card temp

Mykle1

@cowboysdude said in Module Developer Challenge - I surrender!:

He can’t it’s a secret HAHHAHAHAHHA

My code (if you can even call it that) is never a secret. It’s more like an embarrassing teenager moment. ;-)

However, I do enjoy learning and I do appreciate you taking the time to teach me. You’re a damn good dude! :-)

ninjabreadman

@Mykle1 Glad to hear you got it working!

Is this your dev environment? Or do you really have an i7 780, GTX 660 and 16Gb running MM? At least it can run MMM-Globe!

Mykle1

@ninjabreadman said in Module Developer Challenge - I surrender!:

Glad to hear you got it working!

Thanks ninja. Your suggestions pointed me in the right direction, but even then I was at a loss on how to proceed. @cowboysdude deserves all the credit, though. He took the time to explain things to me.

Is this your dev environment? Or do you really have an i7 780, GTX 660 and 16Gb running MM? At least it can run MMM-Globe!

This is my dev machine. It dual boots Windows and ubuntu. I found it on craigslist, although it came with 8GB of RAM. Luckily for me, it wouldn’t boot when I went to buy it so I stole it for $80 and had it up and running in minutes after getting it home.

MMM-Globe! Yes, very nice module. Stunning graphics. But I prefer MMM-EARTH with an animation speed of 0 and a rotate interval of 1000. Also challenging for a Pi but the laptop boards I use handle it nicely and I get the most recent images of Earth. I’ve seen hurricanes develop and an eclipse. Pretty spectacular stuff. :-)

E3V3A

@Mykle1 I love seeing all the progress and work you put into your module development. I must say, as an old time computer fanatic, I’m very impressed with your progress. I’m totally new to MM and still find the node/electron/express framework a rather challenging learning experience. I’ve done some fair amount of rather advanced R&D and bare-metal programming and hacking, and never expected that something based on JS, would be the thing to slow me down! Boy was I wrong and full respect to all those dev’s here, who are able to produce such nice work. I guess it again proves that technical skills without creativity is useless.

As for the Module, IMHO each individual core Temp is not a very interesting parameter, because of load-balancing. Unless you have a faulty core or are explicitly telling your machine to be running certain processes on a single core. In addition, because of how heat is distributed rather evenly throughout the chip dye, you’d never see much difference than a few degrees, so I would rather average the core temps into one, and possibly show the divergence of them. (I.e. the greatest % difference from the average.)