Skip to content

Conversation

@L4co77
Copy link

@L4co77 L4co77 commented Dec 12, 2025

Script that Enhances behaviors.xml with usage information (<UsedIn> elements) showing which Objectives use each behavior.

@L4co77 L4co77 self-assigned this Dec 12, 2025
@L4co77 L4co77 modified the milestone: 8.9.1 Dec 12, 2025
@L4co77 L4co77 force-pushed the Show_example_implementations_usages_of_Behaviors_and_Objectives branch from 7a29fa6 to c41efd6 Compare December 15, 2025 08:55
@L4co77 L4co77 requested a review from davetcoleman December 15, 2025 08:58
@noah-wardlow
Copy link

I dont think is a valid tag for behavior trees. would this be better as a new MetaData field?

@L4co77
Copy link
Author

L4co77 commented Dec 15, 2025

I dont think is a valid tag for behavior trees. would this be better as a new MetaData field?

Just let me know, and I can change it to MetaData.

@L4co77 L4co77 force-pushed the Show_example_implementations_usages_of_Behaviors_and_Objectives branch from c41efd6 to 67a93f0 Compare January 8, 2026 20:00
@L4co77
Copy link
Author

L4co77 commented Jan 8, 2026

@noah-wardlow @davetcoleman could we merge this ?

@noah-wardlow
Copy link

I dont think is a valid tag for behavior trees. would this be better as a new MetaData field?

Just let me know, and I can change it to MetaData.

Let's change it to metadata. I don't want this to break behavior usage if someone tries to use the edited xml

@L4co77 L4co77 force-pushed the Show_example_implementations_usages_of_Behaviors_and_Objectives branch from 66eedee to b42f96c Compare January 8, 2026 22:14
noah-wardlow
noah-wardlow previously approved these changes Jan 8, 2026
@L4co77 L4co77 enabled auto-merge January 8, 2026 22:28
@L4co77
Copy link
Author

L4co77 commented Jan 8, 2026

@noah-wardlow any idea why it's failing ?

@JWhitleyWork
Copy link
Member

I'm seeing this fail on unrelated PRs too. Something in moveit_pro must have changed that causes the lab_sim tests to time out.

@L4co77
Copy link
Author

L4co77 commented Jan 8, 2026

I'm seeing this fail on unrelated PRs too. Something in moveit_pro must have changed that causes the lab_sim tests to time out.

https://github.com/PickNikRobotics/moveit_pro_example_ws/actions/runs/20833576758/job/59853468424?pr=453 it It seems that the problem is caused by changing STUDIO_CONFIG_PACKAGE=lab_sim and STUDIO_HOST_USER_WORKSPACE to MOVEIT_CONFIG_PACKAGE and MOVEIT_HOST_USER_WORKSPACE.

@JWhitleyWork
Copy link
Member

I think in this particular instance that may be the case but I just updated those in main and the lab_sim test timed out. I tried the same locally and got the same result unless I ran the lab_sim integration tests by itself.

@JWhitleyWork
Copy link
Member

@JWhitleyWork
Copy link
Member

Rebase this branch on main and it should fix the environment variables issue.

@L4co77 L4co77 force-pushed the Show_example_implementations_usages_of_Behaviors_and_Objectives branch from 85a6aaa to f08bfd4 Compare January 8, 2026 23:45
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.

5 participants