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

Allow the user to hide the ellipsoid drawn by EllipsoidJoint::generateDecorations() #3445

Merged
merged 1 commit into from
May 4, 2023

Conversation

tkuchida
Copy link
Member

@tkuchida tkuchida commented May 2, 2023

Fixes Issue 1440 in GUI repo

Brief summary of changes

See discussion here.

Testing I've completed

Looking for feedback on...

How to test this without building the GUI from source…

CHANGELOG.md (choose one)

No need to update because minor change for small number of users.


This change is Reviewable

@tkuchida
Copy link
Member Author

tkuchida commented May 2, 2023

@aymanhab Are you able to test this in a GUI build, or perhaps there's a CI artifact that I can use without building the GUI from source? (Here's a test file: arm26_withEllipsoidJoint.txt) 🐇

@adamkewley
Copy link
Contributor

adamkewley commented May 3, 2023

Seems like a straightforward enough PR 👍

@aymanhab
Copy link
Member

aymanhab commented May 4, 2023

The changes look good @tkuchida Thanks for the fix. If you want an artifact of the GUI based on this branch you can create a PR that modifies the continuous-integration.yaml file to checkout the branch tkuchida-hideEllipsoidDecorations instead of main for opensim-core and then you can test the artifact from ci to see if they work as expected. I tried that here opensim-org/opensim-gui#1443 and would appreciate if you test when ci finishes.

@tkuchida
Copy link
Member Author

tkuchida commented May 4, 2023

@aymanhab Thanks for creating the artifact (direct link here, OpenSim-4.5-2023-05-04-ca13bf9-win64). I opened the test file (arm26_withEllipsoidJoint.txt) and toggled the "visible" flag under the EllipsoidJoint's "Appearance" property but the ellipsoid didn't disappear. I then tried manually adding the following to the .osim file:

<EllipsoidJoint name="myEllipsoidJoint">
    ...
    <Appearance>                  <!-- added -->
        <visible>false</visible>  <!-- added -->
    </Appearance>                 <!-- added -->
</EllipsoidJoint>

and no ellipsoid is drawn when the model is opened (here's the file: arm26_withEllipsoidJointHidden.txt). Not ideal but it will work for my student. I think this PR can be merged here but presumably the GUI needs to be patched as well. 🐶

@tkuchida tkuchida changed the title [WIP] Allow the user to hide the ellipsoid drawn by EllipsoidJoint::generateDecorations() Allow the user to hide the ellipsoid drawn by EllipsoidJoint::generateDecorations() May 4, 2023
@aymanhab
Copy link
Member

aymanhab commented May 4, 2023

Agreed, thanks @tkuchida

@aymanhab aymanhab merged commit ffce474 into main May 4, 2023
7 checks passed
@aymanhab aymanhab deleted the tkuchida-hideEllipsoidDecorations branch May 4, 2023 15:44
@tkuchida
Copy link
Member Author

tkuchida commented May 9, 2023

Thanks @aymanhab and @adamkewley

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.

Cyan ellipsoid drawn to visualize EllipsoidJoint cannot be hidden
3 participants