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

Initial py_trees_parser repo setup #1

Merged
merged 31 commits into from
Jan 31, 2025
Merged

Conversation

erichlf
Copy link
Collaborator

@erichlf erichlf commented Jan 22, 2025

This PR removes references to the SAMXL behavior_tree and attempts to make the py_trees_parser independent and open source.

Motivation and Context

To make the py_trees_parser open source some work needed to be done to filter the SAMXL behavior_tree git repo. The filtering allows us to keep the history for the bit related to py_trees_parser and get rid of the history that is not relevant to the the py_trees_parser. However, filtering can only really be done on a per-file/per-directory basis and therefore there are some bits leftover from the behavior_tree that must be manually removed.

Additionally, there are some extra details that needed to be added such as CI and the README needed to be updated.

Changes

  • Filter repo for py_trees_parser
  • Remove leftover behavior_tree bits
  • Add github action (CI)

Type of Changes

  • Breaking change

erichlf and others added 4 commits January 16, 2025 10:32
@erichlf erichlf requested a review from SyZbidi January 22, 2025 07:48
@erichlf erichlf self-assigned this Jan 22, 2025
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from fe4fa87 to 34a14a4 Compare January 22, 2025 07:59
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from 34a14a4 to 54e7acb Compare January 22, 2025 08:05
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from a72d275 to 4b67fcd Compare January 22, 2025 08:34
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch 2 times, most recently from 8d46757 to 2ed9ba6 Compare January 22, 2025 12:36
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from 2ed9ba6 to 8b0f314 Compare January 22, 2025 12:39
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from a1f2cae to 68ed621 Compare January 22, 2025 12:50
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch 5 times, most recently from e4c44d8 to 0b2bf13 Compare January 22, 2025 14:37
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from 0b2bf13 to 57cae04 Compare January 22, 2025 14:40
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from 3864407 to da5d5cf Compare January 22, 2025 15:01
@erichlf erichlf marked this pull request as ready for review January 25, 2025 09:33
@SyZbidi
Copy link
Member

SyZbidi commented Jan 27, 2025

can you add .gitignore?

@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from a26b4ca to bff8fb2 Compare January 29, 2025 07:54
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from 4640484 to f1aeeee Compare January 30, 2025 10:24
@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from f1aeeee to e9dc63a Compare January 30, 2025 13:36
@erichlf erichlf requested a review from dave992 January 31, 2025 08:25
@SyZbidi SyZbidi self-requested a review January 31, 2025 08:44
Copy link
Member

@dave992 dave992 left a comment

Choose a reason for hiding this comment

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

Suggestion: use the awesome new PR and README templates ;)

Only found minor things, looks good to me otherwise.

@erichlf erichlf force-pushed the ORTHO-277/create_bt_parser_repo branch from b8f72f1 to c8e9722 Compare January 31, 2025 12:25
@erichlf erichlf merged commit 36b46e4 into main Jan 31, 2025
1 check passed
@erichlf erichlf deleted the ORTHO-277/create_bt_parser_repo branch January 31, 2025 12:44
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.

3 participants