-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Request for more information #2
Comments
Hi Wim,
|
Filip, Sorry, I looked at the remaining milage, not the total driven. The plug-in does work correct. Is it possible to extend the plug in for the remaining data? Wim |
Thanks Wim, however which data do you want to see... All data seems a lot
and I think they are not all useful.
Op vr 6 nov. 2020 om 19:41 schreef Vondee <notifications@github.com>:
… Filip,
Sorry, I looked at the remaining milage, not the total driven. The plug-in
does work correct. Is it possible to extend the plug in for the remaining
data?
Wim
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD3SGC625KY5FAY7FKRA3NLSOQ7NBANCNFSM4TMSUPQQ>
.
|
I saw on the domoticz forum you wanted to see: doors closed, remainingRangeFuel, remainingRangeElectric, ChargingStatus, |
Not doing the position makes sense.... Here is the log (I removed / crossed out the position and VIN of the car) 2020-11-06 16:05:06.598 (BMW) 7b 22 76 65 68 69 63 6c 65 53 74 61 74 75 73 22 3a 7b 22 76 {"vehicleStatus":{"v |
It's done... I created a branch called "MoreInfo"... Would you be so kind to test it with your car (as my car does not have electric capacity, I cannot do it myself). If you supply feedback, I will either adapt/correct and/or commit it in the main branch. |
That is fast! I will try this evening to test it. Will let you know the results |
Great work. For some reason I had to update to: Domoticz 2020.2 (build 12554) to have it all working. README: pip3 install urllib3 (I needed to use to install urllib) In my own implementation I did add the GEO location. In addition the GEO is used (polled per minute) so if I enter the driveway the lights go on. May I suggest to add |
Done (battery level)! Will also merge to the main branche. |
Filip, Just discovered that the mileage (Fuel) is in fact the Total milage. I saw the value going up during charging of my car... Wim |
Do you mean with "mileage (Fuel)" the "Remaining mileage (fuel)"? Would be strange because I checked the code and for the "Remaining mileage (fuel)" I taken the information 'remainingRangeFuel'... Could you give me two logs together with a screenshot of the device (at the same moment). You can take the screenshot from the Setup - Devices (filter on the BMW devices). Be sure that the screenshot contains all the data. |
Herwith some data: 2020-11-08 18:43:45.519 (BMW) 44 72 69 76 65 72 52 65 61 72 22 3a 22 43 4c 4f 53 45 44 22 DriverRear":"CLOSED" Seems that the 'RemaininRangeFuel' in fact is the total of both EV and Petrol Best regards Wim |
Normally the problem of the remaining mileage (fuel) should be solved based on the information you supplied me. So I subtract the remainingRangeElectric from the remainingRangeFuel. Is this OK? Or do you prefer that I just rename the device "Remaining mileage (fuel)" into "Remaining mileage (total)". This would make more sense for me. Let me know and I will change again. |
hi Filip, thanks for this great plugin, but little problem for me in the log i can see that the connection to bmw server is ok but there are some error 2020-11-11 21:10:15.250 Error: (Mini) 'onMessage' failed 'KeyError'. thanks for your help full log here : 2020-11-11 21:09:54.643 (Mini) Pushing 'onHeartbeatCallback' on to queue |
Hi, I suppose you only have front doors/windows and no rear doors/windows. Correct? |
yepee, thanks, no error now all devices updated now except 'Battery Level' |
I know there is still a problem with the charging level... See the same
topic upwards. Will solve this later... I cannot test it because I have no
electric car... So a bit more complex to find out.
Op wo 11 nov. 2020 21:56 schreef tsrz <notifications@github.com>:
… yepee, thanks, no error now all devices updated now except 'Battery Level'
and 'Charging time'
perhaps it is because my car is a Mini Copper SE ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD3SGC3WHOGJR5RZQFOGIILSPL27ZANCNFSM4TMSUPQQ>
.
|
ok no problem, solve it when you have time if you can |
Found the problem of the not working battery level. In the plugin.py script at row 177 the variable 'ChargingLevelHv' is used, but the capital 'C' must be a normal 'c'. After the change it works now |
Thanks all! Did not had the time to look into detail... One small mismatch
makes a big difference en difficult to see.
Well done.
Will update on GitHub and will close the topic once done. But probably this
weekend.
Filip
Op do 12 nov. 2020 20:01 schreef tsrz <notifications@github.com>:
… perfect, i confirm that it was the solution
[image: batt]
<https://user-images.githubusercontent.com/23282365/98983894-c093eb00-2521-11eb-9934-4105ed2f35c8.JPG>
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD3SGC23M5DEWYNE4NH5L7TSPQWHXANCNFSM4TMSUPQQ>
.
|
Thanks all. Updated and will close the topic. |
Nice plugin. I installed it and is working. I see all kind of information in the log files. As you stated, only the milage is available in Domoticz,
I do would like to have the remaining information available in Domoticz, so frendly request!
Best regards,
Wim
The text was updated successfully, but these errors were encountered: