-
Notifications
You must be signed in to change notification settings - Fork 7
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
Observed frequency is 48.57 Hz rather than 52 Hz (IMU datasheet) #125
Comments
We are probably doing this correctly: https://community.st.com/s/question/0D73W000000bAS4SAM/detail?fromEmail=1&s1oid=00Db0000000YtG6&s1nid=0DB0X000000DYbd&s1uid=0053W000001snzQ&s1ext=0&emkind=chatterCommentNotification&emtm=1684313246967&emvtk=E54ccDtPqWVECsAa5CCiLUY.7M7t1PbWVtlIXpFI4Uk%3D
|
Ouch, +-10% is a lot of uncertainty. But if they say so... Good to keep in mind for future use :) . |
There was a bug in the dataset generation where I forgot to take the offset in the FIFO into account when generating the time vector. Fixed on main. This was a post-processing bug, so by re-running dataset-exports the error is gone and the frequency is much closer to 52 Hz (and no correction is usually necessary). |
I spoke too soon (again...): the variation in frequency is real, but I do not think it is an error on our part. For the buoys tested at sintef the frequencies are:
So within the 10%. The last two buoys are the SINTEF-built ones, with ISM330DHCX. The others are LSM6DSOX. Also they might be from the same batch. |
|
The text was updated successfully, but these errors were encountered: