-
Notifications
You must be signed in to change notification settings - Fork 22
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
Basic Survey Planning #114
Merged
Merged
Conversation
This file contains 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
Includes placeholder directories with hyperlinked `readme.md` files in the documentation hierarchy for the manger, planner, and bridge. Per discussion with @marinagmoreira `survey_manager` lives under `astrobee` to signify the code is intended for cross-compilation.
Adding planning domain, problem, and xml of the atomic actions of the robotic agents
… surveys by Marina - unfortunately causes additional planner flakiness
Attempt at a complete MVP domain model for survey planning
Includes placeholder directories with hyperlinked `readme.md` files in the documentation hierarchy for the manger, planner, and bridge. Per discussion with @marinagmoreira `survey_manager` lives under `astrobee` to signify the code is intended for cross-compilation.
NOTE: If already installed, remove V3 before installing V4
- Remove Ubuntu-16.04 (xenial) CI builds - Update dockerfile ubuntu version defaults to Ubuntu-20.04 (focal) - Update apk build environment to Ubuntu-20.04 (focal)
…em with POPF/OPTIC
Fixes for problem_generator.py
bcoltin
approved these changes
Dec 27, 2023
@marinagmoreira I updated the submodules to use your fixes and made the same change to |
marinagmoreira
approved these changes
Jan 5, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Allows basic planning with the current survey domain.
Remember to update your submodules after pulling. I've found
git pull --recurse-submodules
to be more effective than using the submodule subcommands directly. Also note the submodules undersurvey_planner
have moved and while the new versions should be created auto-magically the old ones could need a manual deletion. I recommend acatkin clean
before this build if you've done anything on the survey branch up to this point and if you haven't you'll probably need tosudo apt install libreadline-dev ros-noetic-behaviortree-cpp
Note this includes @trey0's #112 since I needed the updated survey domain.
Playing with the planner:
roslaunch survey_planner survey_domain.launch
which will setup the required plansys2 nodesrosrun plansys2_terminal plansys2_terminal
to get a repel. Typehelp
for a list of commands with basic tab completion support.@marinagmoreira once the action nodes are registered you can fire them off manually from the palnsys2_terminal REPL with
run action <action_name>
even if there is no plan.