Skip to content
This repository was archived by the owner on May 11, 2025. It is now read-only.

Feature/error check#84

Open
kennyL07 wants to merge 10 commits intodevelopfrom
feature/ErrorCheck
Open

Feature/error check#84
kennyL07 wants to merge 10 commits intodevelopfrom
feature/ErrorCheck

Conversation

@kennyL07
Copy link
Contributor

Pull Request Details

Error Code check and report

Description

Reports if an error occurs

Related Issue

REVLibError does not print/record errors

Which does this affect?

All REVLibError applicable objects.

Testing Done

Checklist

  • My code follows the code style of this project.
  • My code has been tested on the Robot OR includes simulation code to prove functionality.

motor.setSmartCurrentLimit(config.getCurrentLimit());
motor.setIdleMode(config.getIdleMode());
motor.setOpenLoopRampRate(config.getOpenLoopRampRate());
int errorCode = motor.restoreFactoryDefaults().value;
Copy link
Contributor

Choose a reason for hiding this comment

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

A comment here might help for clarity

@azvanderpas
Copy link
Contributor

Use this form of reportError instead of the one having you pass null as the second parameter

reportError​(String error, boolean printTrace)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants