visionspacetec / prust Goto Github PK

This issue is for improving the test framework currently and adding an emulator software to make testing easier and device independent even if it is just for some use cases.

• Add more unit tests for the packet data structures.
• Add qemu for stm32 to test on an emulator.

This finding is related to the following TM[1,7] packet from wiki page:

The packet recieved (in bytes):
[8, 42, 192, 0, 0, 14, 32, 1, 7, 0, 0, 0, 42, 0, 0, 24, 42, 192, 0, 0, 0]

it looks to me, that the last 2 bytes 0, 0 shouldn't be there. From the printed parsed structure, i see, that maybe those bytes represent the field packet_error_control: 0. However, from what i can see in PUS documentation, there is no such field in TM[1,7].

What do you think?

• Moving from the https://github.com/syelman/stm32l4xx-hal fork to https://github.com/stm32-rs/stm32l4xx-hal.
  Since it now has the features we want and it is more up to date

Hello, my name is Kirill! I work with TUDSaT team on a PUS implementation for our CubeSat. You might remember me from OSCSW 2020.
We start using Prust as a reference for our testing. Our test-cases revealed a contradiction between our implementation and Prust. We want to share with you our observations in a series of Github "issues". These are not bug-reports, but rather proposals for discussions, to figure out, what it the right way to do PUS.

In our test-case, we try to decode the example TM[1,7] packet from Prust wiki.

The issue we faced is in PUS TM packet field "absolute time" (ref. PUS document ECSS-E-ST-70-41C from 15 April 2016, Chapter 7.4.3.1). We did not yet decide, which format we want to use - PUS supports several (ref. Chapter 7.3.10). However, none of them is 2-byte long, as used by Prust, according to the example packet.

Did you find a way to decode absolute time in 2 bytes, or is it a overlook? I would be glad to reduce packet size in this field, so, if you found a way - I would be interested to know, how to achieve it