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

SignalReceiver PR #88

Open
wants to merge 153 commits into
base: main
Choose a base branch
from
Open

SignalReceiver PR #88

wants to merge 153 commits into from

Conversation

paulgrammy
Copy link
Collaborator

Features:

  • SignalReceiver is a base of RemoteControl

  • If position and orientation information have been received through settingsCallback, the app will save them in a list of traffic participants

  • this list is later processed while the robot is driving

  • once the robot points towards and is close to the given position of the IE, it sends the last color sent by the IE through SMP to RU

  • designed to work alongside the DCS app EndlessRun

Sorry again for the big PR

Copy link
Collaborator

@gabryelreyes gabryelreyes left a comment

Choose a reason for hiding this comment

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

@BlueAndi @KaiGlaesker @NTDavidS Looks good to me. Now there are only files relevant to the Application. Style and documentation should be mostly correct now.

@BlueAndi
Copy link
Owner

BlueAndi commented Feb 7, 2024

@BlueAndi @KaiGlaesker @NTDavidS Looks good to me. Now there are only files relevant to the Application. Style and documentation should be mostly correct now.

For me its ok as well, my focus was mainly that other Apps were not touched.

@BrndSchndr BrndSchndr self-requested a review February 9, 2024 08:09
lib/SignalReceiver/src/App.cpp Show resolved Hide resolved
lib/SignalReceiver/src/App.cpp Show resolved Hide resolved
lib/SignalReceiver/src/App.cpp Show resolved Hide resolved
lib/SignalReceiver/src/CoordinateHandler.cpp Show resolved Hide resolved
lib/SignalReceiver/src/CoordinateHandler.cpp Outdated Show resolved Hide resolved
lib/SignalReceiver/src/CoordinateHandler.cpp Outdated Show resolved Hide resolved
Copy link
Collaborator

@BrndSchndr BrndSchndr left a comment

Choose a reason for hiding this comment

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

Resolved all comments

@BlueAndi
Copy link
Owner

@paulgrammy Can you fix the conflicting files?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants