Ticket 386

Summary:	IPMI plugin: error in math
Product:	Slurm	Reporter:	Thomas Cadeau <thomas.cadeau>
Component:	Accounting	Assignee:	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,