Initial version of client

[hikirsch]

No description provided.

[choweth]

nitpick: I don't think you need this as they're all defined in your constructor.

[choweth]

nitpick(cont.): and you can still type them here

Suggested change

	self._ip_address = ip_address
	self._port = port
	self._retry_attempts = retry_attempts
	self._command_delay_sec = command_delay / ONE_SECOND
	self._socket_timeout_sec = socket_timeout / ONE_SECOND
	self._ip_address: str = ip_address
	self._port: int = port
	self._retry_attempts: int = retry_attempts
	self._command_delay_sec: float = command_delay / ONE_SECOND
	self._socket_timeout_sec: float = socket_timeout / ONE_SECOND

[choweth]

nitpick: imo, in-line comments are better with this simple formatting. I would use doc strings for functions so that you can parse the info if you ever care to.

Suggested change

	# convert the volume into a usable value. the device will transmit a number
	# between 196 - 255. if it's at max volume, the raw volume will come as 0.
	# this is probably because the gateway only transmits 8 bits per byte.
	# 255 is 0b11111111. since there's no volume = 0 (use mute I guess), if the
	# volume hits 0, it's because it's at max volume, so we make it 256.
	# credit for this goes to lounsbrough
	def convert_volume(raw_volume: int) -> (int, int):
	def convert_volume(raw_volume: int) -> (int, int):
	"""
	convert the volume into a usable value. the device will transmit a number
	between 196 - 255. if it's at max volume, the raw volume will come as 0.
	this is probably because the gateway only transmits 8 bits per byte.
	255 is 0b11111111. since there's no volume = 0 (use mute I guess), if the
	volume hits 0, it's because it's at max volume, so we make it 256.
	credit for this goes to lounsbrough
	"""