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.

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

Symson logging from mbriday #60

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

simson
Copy link
Contributor

@simson simson commented May 1, 2020

No description provided.

@simson simson requested review from mbriday and rixnco May 1, 2020 13:40
@simson simson force-pushed the symson_logging_from_mbriday branch 3 times, most recently from e9415b7 to b487ca3 Compare May 1, 2020 13:45
Mikaël BRIDAY and others added 2 commits May 1, 2020 17:48
(cherry picked from commit 72b51e6)
(cherry picked from commit f143099)
(cherry picked from commit b487ca3)
@simson simson force-pushed the symson_logging_from_mbriday branch from b487ca3 to 3823a6f Compare May 1, 2020 15:49
@simson simson marked this pull request as draft May 1, 2020 18:29
Copy link
Contributor

@rixnco rixnco left a comment

Choose a reason for hiding this comment

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

Need to better define the need for this kind of logging

@mbriday
Copy link
Contributor

mbriday commented May 2, 2020

This log allows to view the times and periods of tasks and interruptions (only tasks for now). The objective is to visualize the Gantt diagram of task/isr execution, in order to detect potential real-time errors. A dump is output on the debug serial line (from hmi request). This dump will be feed to an external tool for visualization.

@mbriday
Copy link
Contributor

mbriday commented May 2, 2020

2 points.

  • log can be disabled at compile time: no computation nor memory usage
  • the visualization tool (simple python script) should give sth like:
    exemple gantt
    Here, this a random set of data for test only.

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.

3 participants