Multithreading Interface Documentation #90
Merged
+246
−70
Conversation
This file contains 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
ClayJay3
added
documentation
|
Added Jesse as a reviewer since the Drone Autonomy Code will be based off of Autonomy_Software - Sprint 0 |
Byrdman32
requested changes
Aug 15, 2023
|
Also, make sure to merge development now that the #89 has been merged. |
|
@ClayJay3 Donovan says that it looks good and approves the PR. |
|
@ClayJay3 Jesse says that it looks good and approves the PR. |
|
Why don't they just approve it? |
|
I don't have the correct permissions. Eli is gonna fix it tomorrow. |
|
Because GitHub is being stupid and wasn't letting Jesse approve it. And Donovan didn't have access to his computer when he reviewed it and asked me to mark it for him. |
wiidler
reviewed
Aug 18, 2023
wiidler
approved these changes
Aug 18, 2023
MRDT-Software
pushed a commit
that referenced
this pull request
May 14, 2024
Multithreading Interface Documentation
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.
Threading Docs
Added a detailed guide on how developers should use the
AutonomyThreadinterface. It goes over how to actually inherit the class and implement the pure virtual functions, as well as some basics on how resources should be handled with mutexes, locks, and atomics.README is in the
src/threadsfolder.A thorough readthrough from a few different people would be nice to catch any errors/inaccuracies or make any suggestions about the formatting.