Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add profiler CLI command #488

Merged
merged 5 commits into from
Feb 26, 2024
Merged

Add profiler CLI command #488

merged 5 commits into from
Feb 26, 2024

Conversation

duduita
Copy link
Contributor

@duduita duduita commented Feb 19, 2024 •
edited
Loading

Summary

  • Only write the profiling data after a header is found, to avoid corrupted files due to the potential residual data in the UART serial
  • If another header is found, start to write a new file, to avoid corrupted files as well (e.g. device resetting while profiling data)

@duduita duduita merged commit fd3bb94 into develop Feb 26, 2024
1 check passed
@stevenkuhn stevenkuhn deleted the add-profiler-cli-command branch February 27, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrianstevens adrianstevens adrianstevens approved these changes

@halyssonJr halyssonJr Awaiting requested review from halyssonJr

@alexischr alexischr Awaiting requested review from alexischr

@ctacke ctacke Awaiting requested review from ctacke

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duduita @adrianstevens