- From: Kostiainen, Anssi <anssi.kostiainen@intel.com>
- Date: Mon, 16 Jun 2014 11:26:38 +0000
- To: Frederick Hirsch <w3c@fjhirsch.com>
- CC: W3C Device APIs WG <public-device-apis@w3.org>, "timvolodine@google.com" <timvolodine@google.com>, Mounir Lamouri <mounir@lamouri.fr>, "cathy.chan@nokia.com" <cathy.chan@nokia.com>
On 11 Jun 2014, at 18:07, Frederick Hirsch <w3c@fjhirsch.com> wrote: [...] > I must admit I’m confused - why is it correct to have the default attribute values emulate a fully charged battery? Shouldn’t the default values be ‘unknown’ until the state of the battery is determined and the values can be properly initialized? I believe this requires more than a note, instead a new 2nd paragraph explanation in section 6 of this model. Added a more elaborate clarification and dropped the note (it is incorporated in the new section): https://dvcs.w3.org/hg/dap/rev/4c334400e0aa > I think Tim is correct, the section on multiple batteries (section 6.1) makes a number of assumptions on whether batteries charge/discharge in serial or parallel so the language and rules may not be correct in all implementations. I suggest we make some of the language non-normative as I propose here (without the formatting): > > [[ > > If a hosting device contains more than one battery, BatteryManager SHOULD expose an unified view of the batteries. > > The charging attribute MUST be set to true if at least one battery's charging state as described above is true. Otherwise, it MUST be set to false. > > The Level attribute can be set to the sum of the levels of batteries of same capacity, or the weighted average of the battery level attributes for batteries of different capacities. > > Depending on whether multiple batteries charge in parallel or serially, the chargingTime attribute can be set to the maximum or sum of the individual battery charging time, respectively. > > The disChargingTime attribute can be set to the sum or weighted average of the individual battery dischargingTime, depending on whether they discharge serially or in parallel respectively. > > ]] Thanks for the suggested changes, updated the spec accordingly: https://dvcs.w3.org/hg/dap/rev/8a9320b0e7d1 > Have all of Tim’s other comments (e.g with respect to calling updateTimer been addressed - I think so) I’ll handle Tim’s other comments separately. Thanks, -Anssi
Received on Monday, 16 June 2014 11:28:00 UTC