Skip to content

Remove rj_rrt as a package - #2542

Open
N8BWert wants to merge 2 commits into
ros3from
rrt-removal
Open

Remove rj_rrt as a package#2542
N8BWert wants to merge 2 commits into
ros3from
rrt-removal

Conversation

@N8BWert

@N8BWert N8BWert commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Description

This PR removes the rj_rrt package. We are currently working on a large rewrite where we remove a good portion of our old code to "clean up" the codebase. Right now rj_rrt may be functional, but it is not used by any of our currently used code so it is slated for removal.

Review Checklist

  • Docstrings: All methods and classes should have the file appropriate docstrings which follow the guidelines in the "Contributing" page of our docs.
  • Remove extra print statements: Any print statements used for debugging should be removed
  • Tag reviewers: Tag some people for review and ping them on Slack

Comment thread install/setup.bash Outdated
COLCON_CURRENT_PREFIX="/opt/ros/humble"
_colcon_prefix_chain_bash_source_script "$COLCON_CURRENT_PREFIX/local_setup.bash"
# setting COLCON_CURRENT_PREFIX avoids determining the prefix in the sourced script
COLCON_CURRENT_PREFIX="/home/n8w3rt/robocup/nate-robocup-software/install"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

prolly should uncommit this

@N8BWert N8BWert Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, this file shouldn't be tracked at all.

Comment thread install/setup.zsh Outdated
COLCON_CURRENT_PREFIX="/opt/ros/humble"
_colcon_prefix_chain_zsh_source_script "$COLCON_CURRENT_PREFIX/local_setup.zsh"
# setting COLCON_CURRENT_PREFIX avoids determining the prefix in the sourced script
COLCON_CURRENT_PREFIX="/home/n8w3rt/robocup/nate-robocup-software/install"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this too

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

👍

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.

2 participants