Skip to content

Conversation

@sokosam
Copy link
Contributor

@sokosam sokosam commented Jun 16, 2025

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't think we need a tuning menu, the driver should just be able to go

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ui looks good

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we shouldn't have a separate script. this should be handled entirely in Python.

https://cantools.readthedocs.io/en/latest/

import can

can_bus = can.interface.Bus('vcan0', bustype='socketcan')
data = example_message.encode({'Temperature': 250.1, 'AverageRadius': 3.2, 'Enable': 1})
message = can.Message(arbitration_id=example_message.frame_id, data=data)
can_bus.send(message)

Since this is a realtime system, we should send the tuning message periodically instead of just once.

Here's a pattern you could try:

  1. Start a thread which sends the Tuning Message every 500 ms.

  2. From the UI, when the user clicks "Submit", change the message that the thread is sending.

  3. Add a toggle switch to enable/disable sending the message

commit 5e5e388
Author: sokosam <samuelshiwork@gmail.com>
Date:   Sat Jun 14 22:19:06 2025 -0400

    dashboard tuning interface

commit e5f995e
Author: sokosam <samuelshiwork@gmail.com>
Date:   Sat Jun 7 14:44:41 2025 -0400

    change states

commit 4137d82
Author: sokosam <samuelshiwork@gmail.com>
Date:   Sat Jun 7 14:33:25 2025 -0400

    dashboard changes

commit 82aed62
Author: Samuel Shi <samuelshiwork@gmail.com>
Date:   Wed May 7 13:59:04 2025 -0400

    tuning
@BlakeFreer BlakeFreer force-pushed the live-tuning-raspberry branch from 8778b55 to 2ce04ad Compare October 8, 2025 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants