Hi Hong-Gunn,<br><br><div><div class="gmail_quote">2012/7/6 Hong-Gunn Chew <span dir="ltr"><<a href="mailto:hglinux@gunnet.org" target="_blank">hglinux@gunnet.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Arnaud,<br>
<div><br>
On Thu, Jun 28, 2012 at 02:36:31PM +0200, Arnaud Quette wrote:<br>
> 2012/6/13 <<a href="mailto:hglinux@gunnet.org" target="_blank">hglinux@gunnet.org</a>><br>
><br>
> > Hi,<br>
> ><br>
><br>
> Hi Hong-Gunn,<br>
><br>
><br>
> > I updated apc-mib.c to use the high precision values, and added<br>
> > input.transfer.reason.<br>
> > I hope you will find the patch useful.<br>
> ><br>
> > <a href="https://alioth.debian.org/tracker/index.php?func=detail&aid=313679" target="_blank">https://alioth.debian.org/tracker/index.php?func=detail&aid=313679</a><br>
> ><br>
><br>
> yup, I saw your patch on Alioth, but had no time yet to acknowledge it,<br>
> sorry.<br>
><br>
> before applying your patch, I need some more validation and information:<br>
> - do "high precision values" complete or replace the existing ones (Ie, are<br>
> the 2 available at the same time)?<br>
<br>
</div>The high precision values are in addition to the existing ones, so both values are available.<br>
<div><br>
> - you've move ambient.*.{high,low} to the first index (Ie, sensor 1) which<br>
> is good since it was planned for the future.<br>
> what puzzles me is that you left ambient.{temperature,humidity}, which<br>
> seems wrong.<br>
> so either there are missing counterpart for ambient.1.*, or the current<br>
> ambient.{temperature,humidity} should be moved to ambient.1...<br>
<br>
</div>I did not move them as they are not for the same probes as the .1. ones.<br>
The first ambient.temperature,humidity is for a built-in ups probe.<br>
The second ambient.temperature,humidity is for an integrated probe in the environmental card.<br>
While the .1. settings are for the external probe 1 of the environmental card.<br>
I can clean this up further if it would be useful, but I didn't do it as I don't have these results.<br></blockquote><div><br>understood.<br>if you anyway have the counterparts for the integrated probe, I would appreciate a patch ;)<br>
<br>I've just modified the namespace to finally expose multiple ambient sensors (r3676).<br>I've further clarified this specific point with r3677:<br><a href="http://trac.networkupstools.org/projects/nut/changeset/3676">trac.networkupstools.org/projects/nut/changeset/3676</a><br>
<a href="http://trac.networkupstools.org/projects/nut/changeset/3677">trac.networkupstools.org/projects/nut/changeset/3677</a><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div>
> - to complete the above and speed up things (read "to catch up my<br>
> late"...), I would appreciate a numeric + a string walk of your APC<br>
> subtree. Please send it in compressed form.<br>
<br>
</div>Ok.  Attached.  And the MIB can be found at<br>
<a href="ftp://ftp.apc.com/apc/public/software/pnetmib/mib/404/powernet404.mib" target="_blank">ftp://ftp.apc.com/apc/public/software/pnetmib/mib/404/powernet404.mib</a><br></blockquote><div> <br>cool, thanks. Anyway, the values I was looking for are missing (well, set to "0"), so that doesn't actually help.<br>
</div></div><br>Part of your patch, there are 2 variables that do not exist yet:<br>- battery.current.total: added since it makes sense<br>- ups.test.date: makes sense too, added with <br><br>Both have been also added with r3676.<br>
<br>Finally, I've applied your patch to the trunk, r3678.<br>Thanks for your contribution.<br><br>cheers,<br>Arnaud<br>-- <br>new blog - <a href="http://arnaud.quette.fr" target="_blank">http://arnaud.quette.fr</a><br>
<br>

</div>