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

Unify utilities used for viewing and plotting between OpenCap, OpenSim, AddBiomechanics #177

Open
jenhicks opened this issue Jul 11, 2023 · 5 comments
Assignees
Labels

Comments

@jenhicks
Copy link
Member

First step is to create a design plan.

@aymanhab
Copy link

@AlbertoCasasOrtiz and @aymanhab met to discuss a pathway for unification and here's an initial plan (considering that node.js modules are the standard for sharing/distributing functionality):

  1. @AlbertoCasasOrtiz will create a node module that encapsulates the charting functionality, this will define the APIs exposed for charting.
  2. Will integrate the module in 1 in opencap and use it in opensim-viewer to plot experimental data, this will allow us to test interoperability (react - vue) and make sure no code duplication moving forward.
  3. Based on the previous 2 steps will repeat for the viewer, and integrate into OpenCap and opensim-viewer and later into AddBiomechanics

@nickbianco
Copy link
Member

@aymanhab, @AlbertoCasasOrtiz, and @jenhicks: I've converted this issue to an epic to organize all issues for discrete development steps towards the unification of these tools. You can associate related issues to the epic above by clicking "Add issues to this epic". I can start by adding a few ongoing issues.

@aymanhab
Copy link

while this is top priority at higher level, I don't think it has specific details that it belongs to sprinting as it stands. Concrete sub-projects are more viable and should be added to sprinting as needed.

@nickbianco
Copy link
Member

@aymanhab, if you don't find this to be useful, feel free to close it. I like these kinds of things to keep projects organized, but it's not the only way to go.

@aymanhab
Copy link

@nickbianco I'm perfectly fine with keeping it, definitely would keep it around as a reminder just not in the sprinting tab/list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants