DIMMs temperature
I want to implement DIMMs temperature, which category should I put it in? fan controllers or new category?

I as a User would expect it in the already existing "RAM" category. Looking forward to have my DIMM Temps shown in LHM! ;)
I tested it with Intel cpu.
does anyone have an amd cpu?
Please help me with this test.
good work!

one thing: should be DIMM 2 and DIMM 4 on my rig instead od DIMM 2 and 3 (if count starts on 1) - otherwise it should be DIMM1 and 3
Could you check the DIMM number with HWiNFO (https://www.hwinfo.com)? thanks in advance.
Hmm its consistent with HWInfo, but CPU-Z shows the right Numbers:

First of all, thank you for finally adding dimm temperatures support! Same issue with DIMM #2 and DIMM #3 here (HWiNFO shows it this way too). MSI X570 motherboard, Ryzen 7 5800X CPU.
smbus address of the dimm sensor is 0x18 to 0x20
your dimm address will be 0x1A and 0x1B.
therefore, index becomes 2 and 3.
you can check it through the System Infomation Viewer program. (http://rh-software.com)

but it doesn't matter. If there is no problem, I will do a pull request soon.
Yes, since you can rename it, it would not be an issue. Thanks again for the work!

I just tried it and a couple of odd things. My DIMMs are named DIMM 4 and 8, which is odd as my motherboard only has 4 slots. The temperature is also a bit odd (given that room temperature is already higher):

EDIT: And thanks for looking into this @lich426, these kinds of additions are appreciated 👍
maybe it's because the dimm chipset is different I don't know all of this either I want someone to modify it after this code is merged
does anyone have an amd cpu? Please help me with this test.


Using a Ryzen 7 5800X
There is a DIMM8 which is not a thing and for some reasons at some point, dimm 2 was on fire. Send me any updated file to test out if you update the code, I have been waiting for DIMM temperature in LHM for so long!
I have already submitted the PR. (#399)
I have already submitted the PR. (#399)
You can still make more commits to your repo, they will be added to the PR. I think fixing the ghost DIMM and possibly the wrong temperature reading would be better done now to ensure the PR goes through
I've given the PR another try and I do not think it's working correctly. My AMD sensors have the wrong number, Intel is probably similary wrong. I think that can be combined with https://docs.microsoft.com/en-us/windows/win32/cimwin32prov/win32-physicalmemory -> the Tag property gives the correct number for me (Physical Memory 1 and Physical Memory 3).
What's another concern is that, unlike the results from @Cubox, my sensors seem to return bogus data. They're certainly not 15 and 5 degrees C. HWiNFO doesn't return anything here either, so I guess the detection isn't working well.
smbus address of the dimm sensor is 0x18 to 0x20 therefore, the index is as follows 0x18 : 0 0x19 : 1 0x1A : 2 0x1B : 3 ...
I only provided a way to get a dimm temperature sensor using a smbus. each chipset of dimm sensors will have a different method of importing temperature sensors. however, I don't know how to handle exceptions for chipsets. I want someone to do elaborate coding. I referred to this datasheet (https://www.onsemi.com/pdf/datasheet/cat34ts02-d.pdf)
Is there a way to distinguish between DIMM with and without a temperature sensor?
It is similar to i2cdetect of Linux.
when looking for a chipset connected to i2c, if the address and value are the same, there is a chipset connected.
ex)

the source code I worked on has SmbDetect method.
Any update on this feature?
Any update on this feature?
The problems mentioned are not resolved yet.
does anyone have an amd cpu? Please help me with this test.
Working well here in FanCtrl. Missing the ability to graph it in LHM.

The problems mentioned are not resolved yet.
Is there any chance we could merge the existing patch which appears to work quite well in some cases, with a view to adding more sensor IC support at a later date? It would be a shame if this were held back entirely until every memory module could be supported.
Having used the dll from the PR since march to add DIMM capability to FanControl, I have nothing to complain about. It does see a phantom 8th stick at 13°C, but I can just ignore it.

Merging the PR would allow everyone to at least try and see if their DIMMs are properly picked up, and use those values until this is entirely fixed.
@Cubox Can you try this one: https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/actions/runs/960007578 I don't seem to have any DIMM with temperature sensor here.
@Cubox Can you try this one: https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/actions/runs/960007578 I don't seem to have any DIMM with temperature sensor here.
I'm afraid this one broke it for me. No temperatures in this version. All of them have been good for me so far otherwise.
Broken for me too, no DIMM Temps with this one.
@Cubox Can you try this one: https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/actions/runs/960007578 I don't seem to have any DIMM with temperature sensor here.
not a single DIMM :(
@xcasxcursex @Eno77 @Cubox Anything with this one? https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/actions/runs/964965830
