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

imu2sto app #2

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

Conversation

raulmartingomez
Copy link

App designed to convert IMU data to quaternions and OpenSim .sto files

@aymanhab
Copy link
Member

Hello, thanks for the contribution. Definitely excited to have this mechanism in place and have you start using it. Some general comments:

  • Generally exe files do not belong to repository, I'd remove from the commit and add a link on the readme file instead
  • Ideally some mention of license or usage terms and support/contact would be helpful.

Thank you

@raulmartingomez
Copy link
Author

Thank you very much for creating this sharing space. I will a link to the .exe file, I will also add another link to the app I just release this week on Google Play to collect IMU data from Movesense devices that can be used with the imu2sto script <link>.

For the license it would be enough if I add to the README file this?:

License

This project is licensed under the MIT License. See the https://opensource.org/license/mit/ for details.

Usage

You are free to use, modify, and distribute this app as long as you include a copy of the license and give credit to the original author. If you make improvements to the app, I encourage you to share them with the community by contributing to this repository.

Support

If you have any questions or issues with the app, please open an issue on GitHub and I will do my best to assist you.

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