Python API "client.SendDmx" priority support?

[taf-dk]

Can I somehow include the priority flag via the python API?

The reason I need it is that I would like to control the light locally via temperature probe and overwrite this with external artnet DMX input.

I guess it should be placed in the client.SendDmx(universe, data, DmxSent) function, looking at the link below.

http://docs.openlighting.org/ola/doc/latest/structola_1_1client_1_1_send_d_m_x_args.html

My code:
code.txt

[peternewman]

Hi @taf-dk ,

You're right we don't currently support it:

ola/python/ola/OlaClient.py

Lines 937 to 965 in e95de59

	def SendDmx(self, universe, data, callback=None):
	"""Send DMX data to the server
	Args:
	universe: the universe to send the data for
	data: An array object with the DMX data
	callback: The function to call once complete, takes one argument, a
	RequestStatus object.
	Returns:
	True if the request was sent, False otherwise.
	"""
	if self._socket is None:
	return False
	controller = SimpleRpcController()
	request = Ola_pb2.DmxData()
	request.universe = universe
	if sys.version_info >= (3, 2):
	request.data = data.tobytes()
	else:
	request.data = data.tostring()
	try:
	self._stub.UpdateDmxData(
	controller, request,
	lambda x, y: self._AckMessageComplete(callback, x, y))
	except socket.error:
	raise OLADNotRunningException()
	return True

I think we'd be able to add it fairly easily, but it would need a new method to deal with the extra parameter SendDmxPriority(universe, data, priority, DmxSent).

Can't you work around it though; the default priority should be followed by the Python stuff (i.e. 100), so can't you just set your Art-Net port to use a higher priority and it will override it when present?

[taf-dk]

Hi @peternewman

Thank you for the quick reply, changing the priority for the artnet input from 100 to 110 input solved the problem for my use case.

[peternewman]

Great, glad to hear that works, I'll leave the issue open so we can add it at some point...