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

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

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

wants to merge 12 commits into from

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.

Still a WIP

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 and others added 12 commits November 25, 2024 14:38
@jturner65
--initial commit
1390b0c 
Utility directory hierarchy; interactive viewer-oriented utility classes
@jturner65
--re-add ortho function import
57a2a2a
@jturner65
--update docs configuration script
367b70c
@jturner65
--verify objects are selected before attempting to match their dims
8de8d39
@jturner65
--properly set matching orientation
d39ca63 
Need to go through the _move_one_object function to appropriately handle undo
@jturner65
--accept 'body' as well as 'root' for baselink name
b89793c
@jturner65
--fix adding object duplicates (should place them right above original)
863d5e8
@aclegg3 @jturner65
update the default_link code to align with new lab util
95717a7
@jturner65
--rollback default link change
89f126e
@jturner65
--clean up imports
7a59050 
Apparently python 3 doesn't like class-scope imports.
@jturner65
--object existing check before access
f892afb
@jturner65
--use new debugLineRenderer function for axes
3f3f59e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aclegg3 aclegg3 Awaiting requested review from aclegg3

At least 1 approving review is required to merge this pull request.

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