Modified the code to work with Octave 3.6.4 in Windows - #2
Open
tjrantal wants to merge 1 commit into
Open
Conversation
|
Wuju, forgive my unprofessional excitement. |
Author
|
Hello Thomas,
I haven't worked with the code in 5 years and am currently using different
code to work with x-io data. What are you trying to achieve? I might be
able to give you a leg up. If it's specifically on position tracking, then
I'm not able to help.
On a related note, I'm on my summer holiday so will be slow to respond.
Regards,
-Timo
…On Tue, 2 Jul 2019, 17:32 Thomas Haberkorn, ***@***.***> wrote:
Hi mate,
Is there any update on this project?
I tried to run script.m on Octave 5.1.0. It fails when loading the
dataset. In Matlab, this step works right of the box.
Do you know how to mitigate this issue?
Kind regards,
Thomas
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=AAIDBEH7JP4SFGXNLWY43FDP5NRHRA5CNFSM4AKSO5U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZBPHFQ#issuecomment-507704214>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAIDBEARQK5MDAJ6PWC2DQLP5NRHRANCNFSM4AKSO5UQ>
.
|
|
For those struggling to reproduce (like me): Make sure to load the packages signal (for butter()) and mapping (for deg2rad()) in Octave 3.6.4. The top of the Script.m file thus has to look like this: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Some of the features in the code weren't implemented in Octave version 3.6.4, so I modified the code so that the Script.m runs through in Octave in Win7.
On a related note, thank you for publishing the code! I'm going to start using an IMU for assessing physical performance as affordable (not to mention more portable and convenient) alternative to a force plate (which is what I've used in the past). Dr. Madgwick's method ought to work as is, and saved me a lot of unnecessary re-inventing the wheel (assuming that I would've ever gotten there, which is not a given). I will cite Dr. Madgwick's thesis and/or the paper indicated in x-io Technologies web page when I publish my work (in a couple of years :) ). Please let me know if you'd prefer some other references to be cited.