Skip to content

ControlBoard improvements #3290

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ControlBoard improvements #3290

wants to merge 1 commit into from

Conversation

randaz81
Copy link
Member

  • getLimits() renamed to getPosLimits()
  • all methods of iControlLimits.h and IPidControl.h now use yarp::dev::ReturnValue
  • improved ControlBoardMsgs

@update-docs Update Docs
Copy link

update-docs bot commented Sep 26, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

@randaz81
* getLimits() renamed to getPosLimits()
7bb0d5d 
* all methods of iControlLimits.h and IPidControl.h now use yarp::dev::ReturnValue
* improved ControlBoardMsgs
@randaz81 randaz81 force-pushed the controlBoard_cleanup branch from b244cc6 to 7bb0d5d Compare September 26, 2025 14:58
@randaz81 randaz81 deployed to code-analysis September 26, 2025 15:00 — with GitHub Actions Active
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
30.0% Coverage on New Code (required ≥ 80%)
15.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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

@Nicogene Nicogene Awaiting requested review from Nicogene Nicogene is a code owner

@traversaro traversaro Awaiting requested review from traversaro traversaro is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Component: Devices Component: Library - YARP_dev Type: Breaking/Behaviour Change Involves breaking user code or behaviour
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@randaz81