-
Notifications
You must be signed in to change notification settings - Fork 184
Position Override created to allow position testing #2263
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
Merged
Merged
Changes from 11 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
f381c84
Added UI
jvogt23 c67d840
Partial copy of main_window changes
jvogt23 0340e89
partial message addition
jvogt23 c44ad84
OverridePosition.msg
jvogt23 5b17f1f
Merge branch 'ros2' into test-mode
jvogt23 dfa6e38
Set up working override positions, needs refactor
jvogt23 8d2c07b
Working position override created
jvogt23 0036b6e
Fix bug where changing the goalie num to a robot that is not auto doe…
jvogt23 907aa11
Fixes based on code review
jvogt23 9de5737
force test rerun
jvogt23 a501d4e
Fix Code Style On test-mode (#2264)
github-actions[bot] c84a54a
Resolving final conversations for review
jvogt23 2673072
Merge branch 'test-mode' of github.com:RoboJackets/robocup-software i…
jvogt23 cd5bffd
Fix Code Style On test-mode (#2315)
github-actions[bot] d86225e
Merge branch 'ros2' into test-mode
jvogt23 3175848
Removed LENGTH enum value
jvogt23 7e4c95d
asdf
jvogt23 bd5bff0
Update soccer/src/soccer/strategy/agent/position/robot_factory_positi…
jvogt23 a217a0e
Fix braces on if statement
jvogt23 d476111
Style fixes
jvogt23 5ca24d4
Made different topic for each robot id's overriding position
jvogt23 4030e05
FixStyle on changes
jvogt23 18ac7b9
Rename topic to fit convention
jvogt23 b150791
Fix naming conventions and comments
jvogt23 64cbc9d
Last thing
jvogt23 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
# Contains two unsigned ints - The robot ID to override and the position | ||
# to set for that ID, based on the enum OverridingPositions, | ||
# which is contained in soccer/src/soccer/strategy/agent/position/overriding_positions.hpp | ||
|
||
uint32 robot_id | ||
jvogt23 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
uint32 overriding_position |
28 changes: 28 additions & 0 deletions
28
soccer/src/soccer/strategy/agent/position/overriding_positions.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
#pragma once | ||
#include <map> | ||
#include <string> | ||
#include <vector> | ||
|
||
namespace Strategy { | ||
/* | ||
OverridingPositions refers to the positions that can be manually set in the UI. | ||
Normally, all robots are set to Auto. If you want to add a new position, add a new value | ||
to this enum before LENGTH, add a string to the overriding_position_labels vector in | ||
main_window.hpp, and add a case to the check_for_position_override method in | ||
RobotFactoryPosition. | ||
*/ | ||
enum OverridingPositions { | ||
AUTO, | ||
OFFENSE, | ||
DEFENSE, | ||
FREE_KICKER, | ||
PENALTY_PLAYER, | ||
PENALTY_NON_KICKER, | ||
SOLO_OFFENSE, | ||
SMART_IDLE, | ||
ZONER, | ||
IDLE, | ||
LENGTH, // Do not remove | ||
jvogt23 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
}; | ||
|
||
} // namespace Strategy |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.