ecomode_addon-2

[masterwishx]

add-on for #2637,#2680 ECO/Bypass for Eaton Models

@jimklimov In this new pr we can:

You think we can add ESS stuff also, however
Only enterprise UPSes have it?

• [ Done ] We can add also post limits not passed in debug when check range eco/bypass func.

• [ Done ] added variables:

input.bypass.switchable
input.transfer.bypass.overload
input.transfer.bypass.outlimits

• [ Done ] ESS added to usbhid-ups.c

CC @jimklimov please confirm if it needed .

Also for now can't leave eco mode by commands but maybe Becouse my ussue with timeout, not sure...

[AppVeyorBot]

❌ Build nut 2.8.2.2368-master failed (commit e298656580 by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2369-master completed (commit 813fcec36b by @masterwishx)

[masterwishx]

@arnaudquette-eaton Very sorry to bother you again ,maybe you can find info about:

UPS.PowerConverter.Input.[2].Switchable

[arnaudquette-eaton]

no need to be sorry @masterwishx , I'm always happy to help ;)

Desc: Item of Bypass AC Input collection
Transfer on automatic Bypass enabled

RW

0: The automatic transfer on bypass is not allowed
1: The automatic transfer on bypass is allowed. (Refer to UPS.PowerConverter.Input.[2].OutOfToleranceTransferEnable and UPS.PowerConverter.Input.[2].OverloadTransferEnable for additional bypass rules)

Hope it helps

[masterwishx]

Transfer on automatic Bypass enabled

Thanks a lot , so its like UPS.PowerConverter.Input.[2].ForcedTransferEnable but like for UPS itself bypass autoswitching with rules?

as you know we now have ECO/Bypass enabled with your help for HID info!

the only problem i cant exit from ECO to bypass by r/w values or commands bypass.on/off , but maybe its local issue for my UPS with timeouts ....

Also will try to add function for auto enter to ECO mode ( bypass.on + ECO + bypass.off) but still cant test exit from ECO :(
Thanks again for your help

[masterwishx]

Transfer on automatic Bypass enabled

added values if you think they usefull ?

[AppVeyorBot]

❌ Build nut 2.8.2.2373-master failed (commit 87e718f5df by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2374-master completed (commit 09443bb338 by @masterwishx)

[masterwishx]

CC @arnaudquette-eaton can you please check and confirm that we can exit from ECO mode by :
"UPS.PowerConverter.Input.[2].SwitchOffControl"=1 input.bypass.switch.off or
"UPS.PowerConverter.Input.[2].SwitchOnControl"=1 input.bypass.switch.on
or maybe some other command ? (cant exit from ECO by this command, but maybe its my local issue ?!?)

[AppVeyorBot]

❌ Build nut 2.8.2.2383-master failed (commit 985fb4d495 by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2386-master completed (commit 72fd7b8e62 by @masterwishx)

[AppVeyorBot]

❌ Build nut 2.8.2.2387-master failed (commit d98bf537da by @masterwishx)

[masterwishx]

IFrom 8afba96
Work for function for auto ECO started , please confirm if we need this at all ? CC @jimklimov @arnaudquette-eaton @desertwitch

Another easy method just to make bash script with (bypass.start,ecomode.enable etc... )

[AppVeyorBot]

✅ Build nut 2.8.2.2389-master completed (commit 594d32a999 by @masterwishx)

[masterwishx]

@jimklimov please confirm here we should call function to check range becouse :
when setvar("input.eco.switchable", "ECO"); its only set variable but not calling function eaton_input_eco_mode_check_range like here, same for bypass?

nut/drivers/mge-hid.c

Lines 1709 to 1710 in 858f615

	/* ECO(HE), ESS Mode switch, to use when 'input.bypass.switch.on' is on */
	{ "input.eco.switchable", ST_FLAG_RW | ST_FLAG_STRING, 8, "UPS.PowerConverter.Input.[5].Switchable", NULL, "%.0f", HU_FLAG_SEMI_STATIC, eaton_input_eco_mode_on_off_info },

nut/drivers/mge-hid.c

Lines 783 to 789 in 858f615

	/* High Efficiency (aka ECO) mode, Energy Saver System (aka ESS) mode makes sense for UPS like (93PM G2, 9395P) */
	static info_lkp_t eaton_input_eco_mode_on_off_info[] = {
	{ 0, "normal", NULL, NULL },
	{ 1, "ECO", eaton_input_eco_mode_check_range, NULL }, /* NOTE: "ECO" = tested on 9E model and working fine */
	{ 2, "ESS", NULL, NULL },
	{ 0, NULL, NULL, NULL }
	};

@arnaudquette-eaton please confirm that seems setvar also call info_lkp_t *<unnamed>::hid2info
so no need to call this function befor like i wrote above ?

nut/drivers/usbhid-ups.c

Lines 969 to 972 in b624145

	/* Lookup the new value if needed */
	if (hidups_item->hid2info != NULL) {
	value = hu_find_valinfo(hidups_item->hid2info, val);
	} else {

[AppVeyorBot]

❌ Build nut 2.8.2.2392-master failed (commit 97b0d1f4b5 by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2393-master completed (commit d3d37865f4 by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2396-master completed (commit 39078e0566 by @masterwishx)

[masterwishx]

@arnaudquette-eaton Sorry again but can you check and confirm that it was done right ?

UPS.PowerConverter.Input.[5].PresentStatus.Used

info_lkp_t eco_mode_info[] = {
    { 0, "normal", NULL, NULL },
    { 1, "ecomode", NULL, NULL },
    { 2, "essmode", NULL, NULL },
    { 0, NULL, NULL, NULL }
};

or we should use :

info_lkp_t eco_mode_info[] = {
    { 0, "!ecomode", NULL, NULL },
    { 1, "ecomode", NULL, NULL },
    { 2, "essmode", NULL, NULL },
    { 0, NULL, NULL, NULL }
};

[AppVeyorBot]

❌ Build nut 2.8.2.2406-master failed (commit b70396de2e by @masterwishx)

[AppVeyorBot]

✅ Build nut 2.8.2.2415-master completed (commit 2870c688e1 by @masterwishx)