@cryptotaxi247 / netdata-1 / commits / c355e1bb9

Fixed a bug where solar consumption is 100% even though solar power is zero.

Chris committed Jan 29, 2018 at 17:48 UTC c355e1bb984061ec72ffb5a1f03639e03413763c
2 files changed +4 -3
node.d/fronius.node.js
+3 -3
@@ -259,9 +259,9 @@ var fronius = {
259 var selfConsumption = site.rel_SelfConsumption;
260 var solarConsumption = 0;
261 var load = Math.abs(site.P_Load);
262 - var power= Math.max(site.P_PV, 0);
263 - if (load >= power) solarConsumption = 100;
264 - else if (power <= 0) solarConsumption = 0;
262 + var power = Math.max(site.P_PV, 0);
263 + if (power <= 0) solarConsumption = 0;
264 + else if (load >= power) solarConsumption = 100;
265 else solarConsumption = 100 / power * load;
266 return this.getChart(this.getSiteAutonomyChart(service, "autonomy"),
267 [
tests/node.d/fronius.parse.spec.js
+1
@@ -237,6 +237,7 @@ describe("fronius parsing for autonomy", function () {
237
238 it("should return 0 for solarConsumption if PV is null", function () {
239 site.P_PV = null;
240 + site.P_Load = -1000;
241 var result = subject.parseAutonomyChart(service, site).dimensions[2];
242
243 expect(result.name).toBe(subject.solarConsumptionId);