UMSATS / soti

Contains software for the SOTI (Satellite Operations Testing Interface) application, which runs on a NUCLEO-L452 development board.
5 stars 1 forks source link

Improve Send Command Feedback #31

Open Koloss0 opened 2 weeks ago

Koloss0 commented 2 weeks ago

Right now when you send a command through SOTI, you get feedback like this:

Sending bytes to the satellite: 0x1F020321123ABC00EF0000

Which is not super helpful. For one, it's formatted in SOTI's internal representation of messages, so it's not even what's being sent over the CAN lines. And second, most people won't even understand how to read it.

For this issue, change the output to:

Sent command 'PLD_SET_ACTIVE_ENVS' to Payload

Feel free to come up with your own suggestions in the comments below.

ArnavG-it commented 2 weeks ago

It might be useful to store the outgoing messages to a .json file as well (which should be simple because the parsing functions can be generalized) so that the full testing session is recorded with timestamps. What do you think?

Koloss0 commented 2 weeks ago

Ooo. I like your thinking. Yeah, we can record entire sessions, that would be cool. You could then share them with team mates to demonstrate an issue with a component.

I also think if we're gonna use it for this purpose, we should make the formatting easier to read. I think YAML might be a better format over JSON:

{
  "time": "10:56:20",
  "sender": "CDH",
  "recipient": "PLD",
  "cmd": "PLD_SET_ACTIVE_ENVS",
  "priority": 32,
  "body": {
    "bitmap": "0b0000000000000000"
  }
}

Vs.

- time: '10:56:20'
  sender: CDH
  recipient: PLD
  cmd: PLD_SET_ACTIVE_ENVS
  priority: 32
  body:
    bitmap: 0b0000000000000000
Koloss0 commented 2 weeks ago

see #35