bmw_connected_drive - provide position of HA installation - WIP

[ChristianKuehnel]

Description:

This solves issues with some vehicles as discussed in
bimmerconnected/bimmer_connected#62

These vehicles require the client to upload it's current position to the server. Only if this position is within some range (e.g. 2km) of the vehicle, you will get data from the vehicle.

This feature is configurable, so you only need to disclose the location if your car requires it. See example below

⚠️ This is work in progress and needs some testing by someone with a car affected by the problem.
⚠️ This also requires a new release from the underlying bimmer_connected library, once it was tested successfully.

Related issue (if applicable): fixes bimmerconnected/bimmer_connected#62

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>

Example entry for configuration.yaml (if applicable):

⚠️ This feature is only activated if you set send_location: True

bmw_connected_drive:
  5er:
    username: <connected drive username>
    password: <connected drive password>
    region: <your region (as before)>
    send_location: True

Checklist:

• The code change is tested and works locally.
• Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

• New dependencies have been added to the REQUIREMENTS variable (example).
• New dependencies are only imported inside functions that use them (example).
• New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
• New files were added to .coveragerc.

If the code does not interact with devices:

• Tests have been added to verify that the new code works.

[houndci-bot]

line too long (80 > 79 characters)

[houndci-bot]

line too long (105 > 79 characters)

[ajmm]

Hi @ChristianKuehnel
this somehow does not work for me.
If I don't provide send_location: True it gives me the old error:

{
"error" : {
"code" : 500,
"description" : "(SmartPhoneUtil-A-101) Mandatory parameter(s) missed or blank: dlat and dlon are required for BMW vehicles!"
}
}

If I set send_location: True then I get:

Error during setup of component bmw_connected_drive

Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/homeassistant/setup.py", line 145, in _async_setup_component
component.setup, hass, processed_config)
File "/usr/lib/python3.6/concurrent/futures/thread.py", line 56, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/bmw_connected_drive/init.py", line 62, in setup
accounts.append(setup_account(account_config, hass, name))
File "/config/custom_components/bmw_connected_drive/init.py", line 95, in setup_account
cd_account.account.set_observer_position(latitude, longitude)
AttributeError: 'ConnectedDriveAccount' object has no attribute 'set_observer_position'

so maybe REQUIREMENTS = ['https://github.com/m1n3rva/bimmer_connected/archive/master.zip#bimmer_connected==0.5.1'] is loading a wrong version without the set_observer_position?
As I understant that was added to the master?

[ajmm]

@ChristianKuehnel : I had to replace REQUIREMENTS = ['https://github.com/m1n3rva/bimmer_connected/archive/master.zip#bimmer_connected==0.5.1'] with REQUIREMENTS = ['https://github.com/m1n3rva/bimmer_connected/archive/master.zip#bimmer_connected==0.5.2'] to force pip to take the newest version of bimmer_connected from the master branch.
Seems to be working ok now.

[balloob]

Going to close this. You can open it again if a new bimmer release has happened.