Component breaks KNX core integration

[farmio]

hass-becker-component-plus-pybecker/manifest.json

Line 7 in 10b69bc

"xknx==0.18.15"

This replaces the xknx version required by the core KNX integration, rendering it unusable.
Please don't pin-require outdated xknx versions in your component.

See https://community.home-assistant.io/t/secureconfig-error-in-knx-integration/566793

[RainerStaude]

I thought the xknx version from manifest.json requirements (of Becker custom integration) will only be used for this integration. I was not aware it overwrites the xknx version used for HA core.

[farmio]

Great! Maybe see this post (and few following) for an alternative approach that would probably be more future proof: https://community.home-assistant.io/t/yet-another-time-controlled-cover-rflink/72759/111?u=farmio

or here nagyrobi/home-assistant-custom-components-cover-rf-time-based#50

[Mattes83]

Unfortunately the pinning was neccessary. In the current version TravelCalculator has no "_calculate_travel_time"

[farmio]

It was just renamed https://github.com/XKNX/xknx/blob/f5b71ca55cb7609065563b219d705d7666075ec1/xknx/devices/travelcalculator.py#L159

But I think it's still not a good idea to import a 16kLOC library for some side-show proportion calculations.

[RainerStaude]

You guys make me crazy. But that is open source 😉
@farmio I like you care about 16k. That reminds me at the daily fight as HW developer with my SW colleaques. But for a RasPi or even larger system that shouldn't be an issue.
And I tested this change yesterday, but I'm running an previous version of HASS.
I think I will just copy the travelcalculator ...

[RainerStaude]

Done 😄
Hope this works now

[farmio]

Thank you very much! Users of KNX and your component will appreciate it 😃