sebhildebrandt / systeminformation

System Information Library for Node.JS
MIT License
2.65k stars 300 forks source link

Same bank label for two ram strips #894

Closed Aspedm closed 3 months ago

Aspedm commented 4 months ago

Describe the bug The si.memLayout method returns two RAM strips, both with identical bankLabels. The raw request yielded the same information.

Current Output Screenshot 2024-03-07 233633 Screenshot 2024-03-07 233654

Expected behavior This ram strips have 2 and 4 slots. Screenshot 2024-03-07 234049 Screenshot 2024-03-07 234101 Screenshot 2024-03-07 234116 Screenshot 2024-03-07 234127

Environment (please complete the following information):

Aspedm commented 4 months ago

I modified command and added field tag. It's match with physical slots. Get-CimInstance Win32_PhysicalMemory | select DataWidth,TotalWidth,Capacity,BankLabel,MemoryType,SMBIOSMemoryType,ConfiguredClockSpeed,FormFactor,Manufacturer,PartNumber,SerialNumber,ConfiguredVoltage,MinVoltage,MaxVoltage,Tag | fl

DataWidth            : 64
TotalWidth           : 64
Capacity             : 17179869184
BankLabel            : BANK 0
MemoryType           : 0
SMBIOSMemoryType     : 34
ConfiguredClockSpeed : 6400
FormFactor           : 8
Manufacturer         : Kingston
PartNumber           : KF564C32-16
SerialNumber         : 5******4
ConfiguredVoltage    : 1400
MinVoltage           : 1100
MaxVoltage           : 1400
Tag                  : Physical Memory 1

DataWidth            : 64
TotalWidth           : 64
Capacity             : 17179869184
BankLabel            : BANK 0
MemoryType           : 0
SMBIOSMemoryType     : 34
ConfiguredClockSpeed : 6400
FormFactor           : 8
Manufacturer         : Kingston
PartNumber           : KF564C32-16
SerialNumber         : 6******1
ConfiguredVoltage    : 1400
MinVoltage           : 1100
MaxVoltage           : 1400
Tag                  : Physical Memory 3
sebhildebrandt commented 3 months ago

@Aspedm for now I added tag number to the bank to avoid adding an extra property (for windows only). Version 5.22.3 just released.