Skip to content

Improved logger saving mechanism when using the RPC#1002

Merged
GiulioRomualdi merged 5 commits intomasterfrom
savedata_tag
Sep 30, 2025
Merged

Improved logger saving mechanism when using the RPC#1002
GiulioRomualdi merged 5 commits intomasterfrom
savedata_tag

Conversation

@S-Dafarra
Copy link
Copy Markdown
Collaborator

This PR does the following:

  • Adds a reset on the image index number after saving. This allows the frame index to restart from zero after saveData is called.
  • Adds the possibility to add a tag in the name before saving the data. This allows to tag the logs while storing it.
  • Reduces the chance of race conditions between the video, the run and the RPC thread.
  • Applies the clang format to the device sources.

The video is paused when saving, to avoid saving frames in a folder that is being renamed
Improved the use of locks to avoid race conditions when saving frames and renaming the corresponding folder
At the moment we publish only the current timestamp
@S-Dafarra
Copy link
Copy Markdown
Collaborator Author

With ce506c2 I added a status port that, at the moment, publishes only the current timestamp.

This is something we discussed a while ago with @PasMarra

@GiulioRomualdi
Copy link
Copy Markdown
Collaborator

CI failure not related to this PR. Merging! Thank you @S-Dafarra

@GiulioRomualdi GiulioRomualdi merged commit 38bb145 into master Sep 30, 2025
8 of 9 checks passed
@GiulioRomualdi GiulioRomualdi deleted the savedata_tag branch September 30, 2025 05:53
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.

2 participants