<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

</head>

<body dir="ltr">

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">

<p style="margin-top:0;margin-bottom:0"><span style="font-size: 12pt;">Dear Daniele,</span><br>

</p>

<div style="color: rgb(0, 0, 0);">

<div dir="ltr">

<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols">

<p style="margin-top:0; margin-bottom:0">I have some news regarding the Driver: I applied the patch you sent me (<a href="https://github.com/zykh/nut/tree/issue-441" class="x_OWAAutoLink" id="LPlnk291633" previewremoved="true">https://github.com/zykh/nut/tree/issue-441</a>)

 and it works correctly (obviously in Level 5 of Debug I see "missing CR...etc..").</p>

<p style="margin-top:0; margin-bottom:0">As for now there are 2 modification I'd like to suggest you:</p>

<p style="margin-top:0; margin-bottom:0"><br>

</p>

<p style="margin-top:0; margin-bottom:0">- For Online Type UPSs the Megatec protocol describes that the battery voltage is provided in the form of V per Cell, not V per block, but the driver doesn't care because I see 2.21V instead of 36V in UPSC (Battery.voltage).

 I think that this should be corrected so the customer can see the string voltage and not the single Cell voltage (Megatec 0.06).</p>

<p style="margin-top:0; margin-bottom:0"><br>

</p>

<p style="margin-top:0; margin-bottom:0">- About battery low and high guesstimation the formula uses these values:</p>

<p style="margin-top:0; margin-bottom:0"></p>

<div>batt.volt.low = 104 * batt.volt.nom / 120   (for a 12V VRLA --> 10.4V batt.volt.low)</div>

<div>batt.volt.high = 130 * batt.volt.nom / 120   (for a 12V VRLA --> 13V batt.volt.high).</div>

<div>In my opinion these values are not correct (a 12V lead acid battery can be charged up to 13.8V while discharged to 9.6V)</div>

<div><br>

</div>

<div><span style="font-size:12pt">Instead I would suggest:</span><br>

</div>

<div>

<div>batt.volt.low = 100 * batt.volt.nom / 120   <span style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">(for a 12V VRLA --> 10V batt.volt.low)</span></div>

<div>batt.volt.high = 135 * batt.volt.nom / 120   <span style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">(for a 12V VRLA --> 13.5V batt.volt.high)</span></div>

with this correction we have also some "Safe Margin", I mean that more or less all the UPS I tested will charge and discharge the batteries at those values.</div>

<div>I would like also to ask you if for this first time we can send you the sources instead of the Diff patch and for the future we will study how to send it in the format required (if you have any link explaining the diff, etc. please send it, it will be

 useful for me).</div>

<div><br>

</div>

<div>The question from Stefano Pongiluppi (my colleague) has been solved because we'll not use Blazer anymore.</div>

<div>Thanks again for the support!</div>

<br>

<p></p>

<p style="margin-top:0; margin-bottom:0"><br>

</p>

<div id="x_Signature">

<div id="x_divtagdefaultwrapper" dir="ltr" style="">

<p style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">

Best Regards,</p>

<p style=""><span style="font-size:11pt"><u>Gabriele Taormina</u></span><span style="font-size:11pt"> </span><span style="font-size:11pt"> </span><span style="font-size:11pt"></span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">UPS </span><span style="font-size:11pt">Strategic Business Unit</span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Field Application Engineer</span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Phone:     +39 0522/207046</span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Fax:           +39 0522/207005</span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Address:  Via Rodano 1 - Reggio Emilia - 42124 - Italy</span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Email:       </span><span style="font-size:11pt"><a href="mailto:gabriele.taormina@legrand.com" class="x_OWAAutoLink" id="LPNoLP" style="font-size:12pt">gabriele.taormina@legrand.com</a></span></p>

<p style=""><span style="font-size:11pt"><a href="mailto:gabriele.taormina@legrand.com" class="x_OWAAutoLink" id="LPNoLP" style="font-size:12pt"></a></span><span style="font-size:11pt">Website:  </span><span style="font-size:11pt"><a href="http://www.ups.legrand.com/" target="_blank" rel="noopener noreferrer" class="x_x_OWAAutoLink" id="LPNoLP" style="font-size:12pt">www.ups.legrand.com</a></span></p>

<p style=""><span style="font-size:11pt"></span><span style="font-size:11pt">Website:  </span><a href="http://www.legrand.com/" target="_blank" rel="noopener noreferrer" class="x_x_OWAAutoLink" id="LPNoLP" style="font-size:12pt"><span style="font-size:11pt">www.legrand.com</span></a></p>

<span style="font-size:11pt"></span>

<div style="color:rgb(0,0,0); font-family:Calibri,Arial,Helvetica,sans-serif,serif,EmojiFont; font-size:16px">

<span style="font-size:11pt"></span><font size="2" style="font-family:Arial,Helvetica,sans-serif,serif,EmojiFont"><img class="x_EmojiInsert" alt="1506322600142_legrand-vector-logo.png" style="user-select: none;" tabindex="0" data-outlook-trace="F:1|T:1" src="cid:9958b4d7-d907-423c-b7ff-a0d5e1a6e26f"><br>

</font></div>

<br>

<p style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">

</p>

</div>

</div>

<br>

<br>

<div style="color:rgb(0,0,0)">

<hr tabindex="-1" style="display:inline-block; width:98%">

<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Da:</b> Daniele Pezzini <hyouko@gmail.com><br>

<b>Inviato:</b> domenica 29 luglio 2018 00:50<br>

<b>A:</b> Stefano PONGILUPPI<br>

<b>Cc:</b> Gabriele TAORMINA; nut-upsdev@alioth-lists.debian.net; Thierry DESTRUEL<br>

<b>Oggetto:</b> Re: [Nut-upsdev] Adding drivers to NUT?</font>

<div> </div>

</div>

<div class="x_BodyFragment"><font size="2"><span style="font-size:11pt">

<div class="x_PlainText">> The problem with "blazer_usb" driver ("blazer_ser" works correctly) is<br>

> related to the following commands:<br>

><br>

> - "F" and "I": when the KRAULER subdriver check these UPS answers it uses<br>

> wrong constants to check the lenght of the received packets, so these<br>

> commands are considered as "not available" by NUT. In particular, without<br>

> the availability of the "F" command, is not possible for NUT to calculate<br>

> the battery capacity level. In my opinion this is a problem of the driver,<br>

> because our UPSs respect the communication protocol document from Megatec<br>

> and also because the blazer_ser works fine. We have tested different UPSs<br>

> models but the problem is the same.<br>

<br>

OK, if, as I am inclined to think, the cause is simply the lack of the<br>

closing CR, once the GitHub issue I mentioned before is fixed, these<br>

problems should disappear in nutdrv_qx (while we could easily apply<br>

the same set of changes to blazer_usb, or change the expected length<br>

to not consider the closing CR, I don't want to touch it right now, as<br>

it already works with USB devices that don't terminate the Q1 reply<br>

with a CR and these other things are not critical, and I want to keep<br>

an easy way for users to get back to the current working behaviour,<br>

just in case...).<br>

<br>

Please, try nutdrv_qx from this branch:<br>

<a href="https://github.com/zykh/nut/tree/issue-441" id="LPlnk435288" class="x_OWAAutoLink" previewremoved="true">https://github.com/zykh/nut/tree/issue-441</a><br>

A log of the driver started with a debug level of 5 would be useful.<br>

<br>

> - "Q1": in this case the problem is only relative to the debug mode ("short<br>

> answer"); in normal mode it works correctly.<br>

<br>

Are you sure blazer_usb complains in debug mode even with the Q1<br>

reply? What version are you using? Because, unless the USB read fails,<br>

it shouldn't, if the reply only lacks the closing CR...<br>

This should only be a problem with nutdrv_qx (and it should now be<br>

fixed in the branch I just linked).<br>

<br>

> - "battery.voltage high/low: the values used in the formula are not correct,<br>

> indipendently by the UPS used.<br>

<br>

Can you elaborate a bit more on this one?<br>

Do you have any suggestion on how to improve our calculations?<br>

</div>

</span></font></div>

</div>

</div>

</div>

</div>

</div>

<br>

<hr>

<font face="Tahoma" size="2">

<p>Ce message, ainsi que tous les fichiers joints � ce message, peuvent contenir des informations sensibles et/ ou confidentielles ne devant pas �tre divulgu�es. Si vous n'�tes pas le destinataire de ce message (ou que vous recevez ce message par erreur), nous

 vous remercions de le notifier imm�diatement � son exp�diteur, et de d�truire ce message. Toute copie, divulgation, modification, utilisation ou diffusion, non autoris�e, directe ou indirecte, de tout ou partie de ce message, est strictement interdite.

</p>

<br>

<p>This e-mail, and any document attached hereby, may contain confidential and/or privileged information. If you are not the intended recipient (or have received this e-mail in error) please notify the sender immediately and destroy this e-mail. Any unauthorized,

 direct or indirect, copying, disclosure, distribution or other use of the material or parts thereof is strictly forbidden.</p>

</font>

</body>

</html>