upd comments pointing to implementation notes

This commit is contained in:
onyx-and-iris 2026-02-27 19:48:35 +00:00
parent 7597202605
commit fa3964bb56
2 changed files with 2 additions and 3 deletions

View File

@ -7,8 +7,7 @@ from .context import Context
from .help import CustomHelpFormatter from .help import CustomHelpFormatter
app = App(name='bus', help_formatter=CustomHelpFormatter()) app = App(name='bus', help_formatter=CustomHelpFormatter())
# The VBAN protocl does not yet define a packet structure for these bus parameters. # See https://github.com/onyx-and-iris/vban-cli?tab=readme-ov-file#implementation-notes - 1.
# Hopefully that will come in the form of a 'VBAN_VMPARAMBUS_PACKET'.
# app.command(eq.app.meta, name='eq') # app.command(eq.app.meta, name='eq')

View File

@ -47,7 +47,7 @@ def on(
If provided, sets the on state to this value. If not provided, the current on state is printed. If provided, sets the on state to this value. If not provided, the current on state is printed.
""" """
if new_state is None: if new_state is None:
# This doesn't work because the VBAN protocol doesn't send an initial NBS1 packet. # See https://github.com/onyx-and-iris/vban-cli?tab=readme-ov-file#implementation-notes - 2.
# console.out.print(ctx.client.strip[index].eq.channel[0].cell[band].on) # console.out.print(ctx.client.strip[index].eq.channel[0].cell[band].on)
return return
ctx.client.strip[index].eq.channel[0].cell[band].on = new_state ctx.client.strip[index].eq.channel[0].cell[band].on = new_state