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.

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

Signal Generator for Tuning #4

Merged
merged 25 commits into from
Nov 6, 2023
Merged

Signal Generator for Tuning #4

merged 25 commits into from
Nov 6, 2023

Conversation

bsutherland333
Copy link
Collaborator

To make tuning ROSplane easier, we've added the ability to send commands to any level of the controller and wrote a signal generator node to publish commands to that controller. Some launch files have also been written to make launching everything more convenient.

iandareid and others added 25 commits October 19, 2023 12:28
@iandareid
Renamed ControllerInternalsDebug for clarity.
0d04ac0
@iandareid
Added the callbacks for the tuning override.
39b0c3a
@iandareid
Implemented tuning override.
accf114
@bsutherland333
Got tuning_signal_generator compiling and mostly stubbed out
f40c612
@bsutherland333
Finished update_param function
c737ec2
@bsutherland333
Got messages publishing
eca25cd
@bsutherland333
Got signal generation working
457b2f4
@bsutherland333
Added ROS services to signal generator
b73b9d7
@bsutherland333
Implemented service functionality for signal generator
91988ad
@bsutherland333
Formatting on signal generator
b792bd6
@iandareid
Created launch file for tuning.
2d4faba
@bsutherland333
Set topics to publish all the time
bbc7296
@bsutherland333
Added ROS params for default command values
ec90831
@bsutherland333
Added ROS callback for toggle step signal
9371a23
@bsutherland333
Completed step response behavior
35d4824
@bsutherland333
Improved step signal behavior
9d11ef5
@bsutherland333
Suppressed unused params warning for ROS callbacks
0102e0e
@bsutherland333
Renamed dt_hz to publish_rate_hz
f8ced51
@iandareid
Added a simulator launch file for tuning.
2755333
@iandareid
Fixed debug issues and topic name issues.
94b2f39
@bsutherland333
Made triangle and sine signal begin at minimum value
1fb3696
@bsutherland333
Changed to use default-magnitude system instead of center-amplitude
8790853
@bsutherland333
Added logic to check if service calls make sense for signal type
82fbd80
@bsutherland333
Added reset on signal type or output change
7131ded
@bsutherland333
Misc formatting
4c53317
@bsutherland333 bsutherland333 merged commit fe1fe9d into main Nov 6, 2023
2 checks passed
@bsutherland333 bsutherland333 deleted the tuning_update branch November 6, 2023 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iandareid iandareid iandareid approved these changes

Assignees

@iandareid iandareid

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bsutherland333 @iandareid