Are you referring to comment section below? There I state @param ms: integer (in ms) etc. If you are referring to something else, please provide an example of what you are expecting to see as I do not understand the comment.
cmd_air_bubbles_data_broadcast_interval_override(self, ms: int, reset: int = NO_RESET) -> int:
It is necessary to have typing information in commands so the service app can call them correctly Also, putting the typing information in the docstring is good, but it's better to have it in the function so that it can be dynamically interpreted when the code is inspected
I think older classes may not have get functions for everything, but Sarina and others prefer get functions to accessing properties directly so we're trying to create get functions for every collected data.
This (2000) is a minimum for a warning. Alarm if > 2200. Sensor should normally be below 2000 (so not a minimum really). I think we still want a minimum. Not sure if 100 was correct before. Check with Systems.
