Component breaks KNX core integration

[farmio]

https://github.com/RainerStaude/hass-becker-component-plus-pybecker/blob/10b69bc76ca3365ceda94656673516192c70ba4a/manifest.json#L7

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 https://github.com/nagyrobi/home-assistant-custom-components-cover-rf-time-based/issues/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 :wink: @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 😃