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

Updates and publish rate #70

Merged
merged 3 commits into from
Jul 14, 2021
Merged

Updates and publish rate #70

merged 3 commits into from
Jul 14, 2021

Conversation

cosama
Copy link
Contributor

@cosama cosama commented Jul 14, 2021

closes #65

This merge request will change the following things:

I was having some issues with my device with respect to publishing rates and it turned out it was related to #65. The issue was that it seems there are two different channels to get the device to send data asynchronously to the host:

  1. Async output data: This configures the device to send out data at a fixed rate. This is discussed in chapter 6.2.7 and 6.2.8 of the manual. It's rate is defined by vs.writeAsyncDataOutputFrequency(...) and the output types with vs.writeAsyncDataOutputType(...). It can be turned off with 0 == VNOFF. If turned off the ouput frequency has no meaning (I removed the calls).
  2. Data requested explicitly through the BinaryOutputRegister: The publish rate is defined as a fraction of the sensor imu rate. This will be delivered next to the async output.

If both modes are configured, they will also be both delivered and can lead to higher rates than expected. The change I applied, is to reconfigure the device to not use async data output at all and only publish the data requested through the BinaryOutputRegister.

@cosama cosama changed the title WIP: Updates and publish rate Updates and publish rate Jul 14, 2021
Copy link
Owner

@dawonn dawonn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update, and nice touch with the dockerfile.

@dawonn dawonn merged commit 31dc02b into dawonn:master Jul 14, 2021
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.

Published topic frequencies not as expected
2 participants