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

Process-profiling #56

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Process-profiling #56

wants to merge 9 commits into from

Conversation

SZhOU-c
Copy link
Contributor

@SZhOU-c SZhOU-c commented Oct 2, 2024

Added a new ProcessTime struct. Added calculation of the loop time for each thread.

@kUin1
Process-profiling
5ce1407 
Added a new ProcessTime struct. Added calculation of the loop time for each thread.
@SZhOU-c SZhOU-c self-assigned this Oct 2, 2024
kUin1 added 3 commits October 5, 2024 14:44
@kUin1
added log statement
d14affa
@kUin1
fixed bugs
3a3b3be
@kUin1
Fixed another bug
4f9fad0 
The code for processing time is tested on the hardware.
zyunlam
zyunlam requested changes Oct 23, 2024
View reviewed changes
Copy link
Contributor

@zyunlam zyunlam left a comment

Choose a reason for hiding this comment

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

Can you put the logging states into a macro function or #ifdefs to check if it's a debug mode so that we don't log anything that we don't need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zyunlam zyunlam zyunlam requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@SZhOU-c SZhOU-c

Labels
None yet
Projects
MIDAS Software
Status: In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SZhOU-c @zyunlam @kUin1