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

Document drivetrain code #14

Open
jkleiber opened this issue Dec 21, 2022 · 0 comments
Open

Document drivetrain code #14

jkleiber opened this issue Dec 21, 2022 · 0 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@jkleiber
Copy link
Contributor

Summary
It would be good to know what our code does. The C++ code currently has little/no docstrings for each function.

The objective of this ticket is to add docstrings to the header files of the drivetrain code. This includes Drivetrain.h and DrivetrainIO.h. In each header file, each function should have a docstring that looks like this:

/**
 * Summary of what this function does
 *
 * @param arg1 Description of arg1
 * @param arg2 Description of arg2
 * ...
 * @param argN Description of argN
 * @return Description of what returns (and in some cases why)
 */
return_type function_name(data_type arg1, data_type arg2, ..., data_type argN);

So for example:

/**
 * Sets the speed modifier variable used to scale the power of user input.
 * 
 * @param speed_mod New speed modifier to set
 * @return True on success, false on failure
 */
bool SetSpeedModifier(const double &speed_mod);

Work To Do

  • Add docstrings in the prescribed format for the drivetrain code

Verification

  • Inspection

Deferred Work

  • Use doxygen to compile the documentation into an API
@jkleiber jkleiber added the documentation Improvements or additions to documentation label Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants