Ticket 386

Summary: IPMI plugin: error in math
Product: Slurm Reporter: Thomas Cadeau <thomas.cadeau>
Component: AccountingAssignee: Danny Auble <da>
Status: RESOLVED FIXED QA Contact:
Severity: 3 - Medium Impact    
Priority: ---    
Version: 2.6.x   
Hardware: Linux   
OS: Linux   
Site: Atos/Eviden Sites Alineos Sites: ---
Atos/Eviden Sites: --- Confidential Site: ---
Coreweave sites: --- Cray Sites: ---
DS9 clusters: --- HPCnow Sites: ---
HPE Sites: --- IBM Sites: ---
NOAA SIte: --- OCF Sites: ---
Recursion Pharma Sites: --- SFW Sites: ---
SNIC sites: --- Linux Distro: ---
Machine Name: CLE Version:
Version Fixed: Target Release: ---
DevPrio: --- Emory-Cloud Sites: ---
Attachments: patch for correct maths in IPMI plugin

Description Thomas Cadeau 2013-07-28 23:52:07 MDT
Created attachment 360 [details]
patch for correct maths in IPMI plugin

There is a bug in the code that leads to a wrong calculation of dE(n)
dE(n) = (t(n+1)-t(n)) * (w(n-1) + w(n)) / 2
instead of 
dE(n) = (t(n+1)-t(n)) * (w(n) + w(n+1)) / 2

the patch change this.

Thomas
Comment 1 Danny Auble 2013-07-30 09:16:17 MDT
Good Catch!  This will be in the next 2.6 (commit b950ad0f63683ae5e6afbb46e06c29528c89e406).  Please check for +80 char lines.

Thanks,