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

[as2_state_estimator] As2 state estimator unify #119

Merged
merged 2 commits into from
Jan 12, 2023
Merged

[as2_state_estimator] As2 state estimator unify #119

merged 2 commits into from
Jan 12, 2023

Conversation

RPS98
Copy link
Contributor

@RPS98 RPS98 commented Jan 12, 2023

Basic Info

Info Please fill out this column
Issue(s) this addresses -
ROS2 version tested on Humble
Aerial platform tested on -

Description of contribution in a few bullet points

  • Unify as2_state_estimator, as2_state_estimator_plugin_base, as2_state_estimator_plugin_external_odom, as2_state_estimator_plugin_ground_truth and as2_state_estimator_plugin_mocap in as2_state_estimator
  • Remove "as2_state_estimator_plugin_" from plugins names.

@RPS98 RPS98 requested a review from miferco97 January 12, 2023 14:01
@RPS98 RPS98 self-assigned this Jan 12, 2023
@RPS98 RPS98 changed the base branch from main to devel January 12, 2023 14:42
@RPS98 RPS98 merged commit 8e4a9ea into devel Jan 12, 2023
@pariaspe pariaspe deleted the as2_state_estimator_unify branch January 23, 2023 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miferco97 miferco97 Awaiting requested review from miferco97

Assignees

@RPS98 RPS98

Labels
state_estimator
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@RPS98