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

[DRAFT] Added GeoJSONSpawnerROS2 Gem #95

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

Conversation

patrykantosz
Copy link
Contributor

This PR adds GeoJSONSpawnerROS2 component, that allows to connect GeoJSONSpawner Gem with ROS 2 interface. Thanks to that the whole functionality of the GeoJSONSpawner is available in the runtime using standard ROS 2 topics and services.

Usage description (as well as description of the available topics and services) is available in the README.

This PR should be rebased on top of the main branch when #93 PR will be merged.

Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
*additionaly added separated struct with configuration

Signed-off-by: Patryk Antosz <[email protected]>
*code refactored
*added new required field to the json schema (id)

Signed-off-by: Patryk Antosz <[email protected]>
*splited into Editor and Game Components
*added randomization (based on the configuration)
*works in the Editor and in the GameLauncher

Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
*changed used `find` function from AZStd::find to AZStd::unordered_set::find

Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
*ROS2 handlers will be moved to new gem
*added simple state queue to properly handle spawn/despawn requests

Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
Signed-off-by: Patryk Antosz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant