Complete tutorial 5#22
Open
AwesomeTornado wants to merge 9 commits into
Open
Conversation
Contributor
|
@mhmokhtarabadi I think this PR is ready for you. Could you review this PR when you have a chance? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #14 by adding profiling commands and handling.
Requesting review from @mhmokhtarabadi @elsevers @noguchi-takahiro
Notes
Timing statistics before adding slowdown code.

Timing statistics after adding slowdown code.

The slowdown was done by measuring the time between task start and current time, and not exiting until the difference was larger than 50us.
Loop time is the time between task runs. If a task is run every second, the loop time is one second. This is not impacted by task speed, so long as the task finishes before it is run again.
Run time is the actual time it takes a task to run. A slower task will always have a longer time here.
The period of a 10kHz signal is 100us, thus the maximum allowable runtime for a 10kHz task is also 100us.
Self-Review
Approachsection of the issue(s) it is closing? YesReviewer Instructions
Reviewers, please copy and paste a suitable review checklist into your review and answer all questions.
Appendix
This section should be the same for all PRs. Do not edit this section when creating a PR.
Review Checklists
Checklists maintained by the eLev lab for research repositories include:
Standard checklist