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.

Sign up for GitHub

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

Jump to bottom

--[BE] - Python-based utilities #2450

Merged
merged 15 commits into from
Feb 12, 2025
Merged

--[BE] - Python-based utilities #2450

merged 15 commits into from
Feb 12, 2025

Conversation

jturner65
Copy link
Contributor

@jturner65 jturner65 commented Aug 16, 2024 •
edited
Loading

This PR introduces python utilities and a directory hierarchy structure for including those utilities.

This also introduces python utility class objects that can be inserted into a viewer file to facilitate editing and analysis of scenes.

Motivation and Context

How Has This Been Tested

Locally; the included example viewers act as tests for the functionality.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@jturner65 jturner65 requested a review from aclegg3 August 16, 2024 15:02
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 16, 2024
@jturner65 jturner65 changed the title --[BE] - Python-based utilities --[WIP][BE] - Python-based utilities Aug 16, 2024
@jturner65 jturner65 force-pushed the Python_SimUtils branch 3 times, most recently from 3cd59ef to b73b257 Compare August 25, 2024 15:46
@jturner65 jturner65 force-pushed the Python_SimUtils branch 3 times, most recently from 0d131bb to 1567a63 Compare September 4, 2024 18:45
@jturner65 jturner65 changed the base branch from main to dev September 13, 2024 12:44
@jturner65 jturner65 force-pushed the Python_SimUtils branch 2 times, most recently from 61b1c05 to 5f111d7 Compare September 18, 2024 17:14
Base automatically changed from dev to main November 4, 2024 16:35
@jturner65 jturner65 force-pushed the Python_SimUtils branch 2 times, most recently from e360a3b to 20f97c4 Compare November 11, 2024 15:58
@jturner65 jturner65 force-pushed the Python_SimUtils branch 3 times, most recently from 7d2099b to 0c0f847 Compare November 18, 2024 19:01
@jturner65 jturner65 marked this pull request as ready for review November 22, 2024 13:57
@jturner65 jturner65 force-pushed the Python_SimUtils branch 2 times, most recently from ff3c9c3 to 2c5a695 Compare November 22, 2024 17:53
@jturner65 jturner65 force-pushed the Python_SimUtils branch 3 times, most recently from 2e2943a to 8178037 Compare January 15, 2025 14:47
@jturner65 jturner65 force-pushed the Python_SimUtils branch 2 times, most recently from de25e79 to 3413622 Compare January 23, 2025 14:05
@jturner65 jturner65 changed the title --[WIP][BE] - Python-based utilities --[BE] - Python-based utilities Jan 30, 2025
@jturner65 jturner65 force-pushed the Python_SimUtils branch 2 times, most recently from 93b45b2 to edec919 Compare February 10, 2025 14:30
jturner65 and others added 15 commits February 10, 2025 16:48
@jturner65
--initial commit
1608ebb 
Utility directory hierarchy; interactive viewer-oriented utility classes
@jturner65
--re-add ortho function import
00783f8
@jturner65
--update docs configuration script
a69a7f5
@jturner65
--verify objects are selected before attempting to match their dims
7ff4196
@jturner65
--properly set matching orientation
43bbc79 
Need to go through the _move_one_object function to appropriately handle undo
@jturner65
--accept 'body' as well as 'root' for baselink name
02b786d
@jturner65
--fix adding object duplicates (should place them right above original)
5cc54d9
@aclegg3 @jturner65
update the default_link code to align with new lab util
d38c713
@jturner65
--rollback default link change
d7999f9
@jturner65
--clean up imports
3e5b0e4 
Apparently python 3 doesn't like class-scope imports.
@jturner65
--object existing check before access
ace130e
@jturner65
--use new debugLineRenderer function for axes
85232ff
@jturner65
--update various viewer demos with magnum changes
eaf599a
@jturner65
--update object editor class
2758e98
@jturner65
--add object duplicates detection
e372c22
aclegg3
aclegg3 approved these changes Feb 12, 2025
View reviewed changes
Copy link
Contributor

@aclegg3 aclegg3 left a comment

Choose a reason for hiding this comment

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

Thanks for aggregating these utils and packaging them for our users.
Many of these are application editor utils. Some require habitat-lab, but those are marked and not imported by default init.
Some of these (like the spot robot base velocity code) are likely temporary given their bespoke nature, but good to have available for now.

@jturner65 jturner65 merged commit 4299467 into main Feb 12, 2025
7 checks passed
@jturner65 jturner65 deleted the Python_SimUtils branch February 12, 2025 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aclegg3 aclegg3 aclegg3 approved these changes

Assignees

@jturner65 jturner65

@aclegg3 aclegg3

Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jturner65 @aclegg3 @facebook-github-bot