Speed settings

[Arkoxs]

Hi, great job on this HACS integration.
It works for my GS308v3 but I have some issues with the GS105v2 switches.

Would it be possible to add support to change the speed settings on the ports?

I'm currently using a python script to switch between "auto" and "disabled" to turn on different ports. It would be great to control access to ports from HA.

Regards,

[foxey]

@Arkoxs Are you still interested in this feature?
Are you able to contribute by creating a pull request for the Python library that implements the API to the switches?

[Arkoxs]

@foxey, I'm still interested in this feature but not sure if I'm able to help. Lack of time and experience in Python.
I'll give it a try though.

[foxey]

I'd like to keep the library as generic as possible, so it would imply that we need to implement it for all models. If you are a very inexperienced developer, this change is not trivial I'm afraid.

[Arkoxs]

Hi @foxey, do you have time to answer some questions about the parsers.py?

[foxey]

Sure, what do you want to know?

[Arkoxs]

Some classes have parse_xxxx defined and some don't.

The class GS108Ev3 is an example, nothing is defined in the class except the init but it works fine.
The class GS105Ev2 is identical but that one results in an error:
File "/usr/local/lib/python3.13/site-packages/py_netgear_plus/parsers.py", line 215, in parse_switch_metadata switch_serial_number = get_first_text(tree, '//table[@id="tbl1"]/tr[3]/td[2]') File "/usr/local/lib/python3.13/site-packages/py_netgear_plus/parsers.py", line 38, in get_first_text raise NetgearPlusPageParserError(message) from error py_netgear_plus.parsers.NetgearPlusPageParserError: XPath //table[@id="tbl1"]/tr[3]/td[2] not found.

Where and how are these switch info pages parsed?

[foxey]

It works with Class inheritance, so GS108Ev3 is using the methods defined in the parent class (PageParser in this case). I didn't have sample pages for the GS105Ev2 model and therefore assumed (wrongly) that the pages were identical to the other GS105/8 models. Somebody open an issue (#90) for that problem and has provided me with the pages I need to update the parser. After that, the GS105Ev2 class definition will probably has its own parse_switch_metadata class that overrides the one defined in PageParser. Does that clarify the situation?