AlexOwl / jtechdigital-ha

Home Assistant integration of J-Tech Digital HDMI Matrix.
3 stars 2 forks source link

Issue with invalid device info #4

Open ryanmsteed opened 7 months ago

ryanmsteed commented 7 months ago

Hey gang, I am able to add the repository and create the integration, but it looks like no entities are being created. This is actually Orei branded matrix, but I'm told they're the same. So I felt that was worth mentioning. Let me know if you need any more details. Thanks for the help!

image

Logger: homeassistant.components.media_player Source: helpers/entity_platform.py:730 integration: Media player (documentation, issues) First occurred: 12:13:26 PM (12 occurrences) Last logged: 12:19:27 PM

jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-module-C88F7', 'default_name': 'hdmioutput1', 'identifiers': {('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7_output_1')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-module-C88F7', 'default_name': 'hdmioutput2', 'identifiers': {('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7_output_2')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-module-C88F7', 'default_name': 'hdmioutput3', 'identifiers': {('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7_output_3')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-module-C88F7', 'default_name': 'hdmioutput4', 'identifiers': {('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7_output_4')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_6C:DF:FB:0C:88:F7')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information.

alejandro5x commented 7 months ago

Same error here: Logger: homeassistant.components.media_player Source: helpers/entity_platform.py:755 integration: Media player (documentation, issues) First occurred: 6:22:55 PM (4 occurrences) Last logged: 6:22:55 PM

jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-Module', 'default_name': 'hdmioutput1', 'identifiers': {('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX_output_1')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-Module', 'default_name': 'hdmioutput2', 'identifiers': {('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX_output_2')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-Module', 'default_name': 'hdmioutput3', 'identifiers': {('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX_output_3')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information. jtechdigital: Not adding entity with invalid device info: Invalid device info {'configuration_url': 'http://IP-Module', 'default_name': 'hdmioutput4', 'identifiers': {('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX_output_4')}, 'manufacturer': 'J-Tech Digital', 'model': 'HDP-MXB44D70M', 'sw_version': 'V1.08.08', 'via_device': ('jtechdigital', 'jtechdigital_XX:XX:XX:XX:XX')} for 'jtechdigital' config entry: device info needs to either describe a device, link to existing device or provide extra information.

jcl9008 commented 3 months ago

Hello - was looking through the code here... The device info setup in media_player.py needs to be updated in order for this to function correctly. You will need to replace default_name= with name= as below:

name=output_info.name if output_info else f"Output {self._output_index}",

alejandro5x commented 3 months ago

Hi @jcl9008 I already created a pull request for this at https://github.com/AlexOwl/jtechdigital-ha/pull/5/files