Skip to content

Visualize scholzpath - #1662

Open
aymanhab wants to merge 267 commits into
mainfrom
visualize_scholzpath
Open

Visualize scholzpath#1662
aymanhab wants to merge 267 commits into
mainfrom
visualize_scholzpath

Conversation

@aymanhab

Copy link
Copy Markdown
Member

Fixes issue #1632

Brief summary of changes

Handle visualization of ScholzGeometryPath instances, first draft that doesn't handle actual wrapping pending changes in opensim-core.

Testing I've completed

Loaded model with Path and it visualizes properly.

CHANGELOG.md (choose one)

  • no need to update because WIP

aymanhab added 30 commits May 1, 2025 23:32
print out url for debugging
Use same process to build visualizer app on mac, linux
* Bare bones react app with websocket communication to the gui, served from top level folder 127.0.0.1:8002/index.html

* GUI side support for updated visualizer, particularly meta data for json file, scale in mm, models with muscles visualize correctly

* Use UserData to communicate opensimType to visualizer

* GUI side of latest viewer wiring

* Remove unused command

* Add opensim-viewer repo as submodule

* Use opensim-viewer instead of threejs folder

* remove extra nested folder use same CMake min version as core

* build react app in GUI build script

* Add submodule

* update submodule name

* Update continuous-integration.yml

use branch so submodule is found

* fix submodule url

* Update continuous-integration.yml

unset CI to troubleshoot npm build

* move opensim-viewer folder under opensim in distro

* Fix path to server root

* start page url change

* Update JettyMain.java

print out url for debugging

* Remove threejs folder/submodule

* Revert "Remove threejs folder/submodule"

* use new branch on opensim-visualizer to keep old builds working

* commit opensim-viewer module

* Build viewer on mac

* Update build script to move assets folder to top level

* Place assets one level up for easy access

* Attempt fix mac build

* Another attempt at copying assets folder rather than contents

* Move serverRootDir

* Remove extra copying of assets and fix linux build

* Update build.xml

Use same process to build visualizer app on mac, linux

* update viewer module
aymanhab and others added 25 commits March 6, 2026 16:25
* Update opensim-viewer module commit

* update viewer again

* Update checkout reference to opensim_46_fixes
* Use new splash png file

* Update frame.gifs
#1599)

* Update continuous integration script for JAR copying platform-specific

* Update continuous integration workflow to remove branch refs

Removed specific branch references for opensim-gui checkout.

---------

Co-authored-by: Ayman Habib <ahabib@stanford.edu>
* Update viewer commit

* splash.gif

* Windows icons for installer and taskbar

* Update icon in Mac resources folder

* new osx icons

---------

Co-authored-by: Ayman <ayman@Aymans-MacBook-Air.local>
* Upgrade Python version and NumPy in CI workflow
* Update CI workflow for JDK and Netbeans versions linux
* netbeans17 on ubuntu
* Updated build process to include listing of module directories
* zip modified distro then upload
* Move opensim.zip to GitHub workspace on ubuntu for uploading

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
* Update viewer commit with recording fix
* Update opensim-visualizer commit to update welcome page to 4.6
* Change url for Doxygen
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* send message to viewer to update animationLoop behavior from GUI
* Update viewer commit with fixes for video recording
Use high-res splash screen
Turn off looping and disable controls while recording
Handle switching between forward and reverse play
Disable looping while in reverse mode
Disable motion slider knob while animation is playing
* Merge all changes to opensim_46 branch, some of these shouldn't be merged to main (those in CMakeLists and ci scripts  that were made to disable caching and to force artifacts to not have a date-commit-hash)
@aymanhab

Copy link
Copy Markdown
Member Author

Once main branch is updated in PR #1661 there should be a single file change to review.

@aymanhab
aymanhab changed the base branch from opensim_46 to main July 23, 2026 21:29
aymanhab added 2 commits July 23, 2026 14:35
Updated CMake build configurations to remove 'Release' in the config parameter.
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.

2 participants