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

Refresh frequency #8

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Refresh frequency #8

wants to merge 7 commits into from

Conversation

jjehl
Copy link

@jjehl jjehl commented Apr 24, 2017

Better to keep at least 25 Hz to avoit jerky motion.

@show0k
Copy link
Member

show0k commented May 3, 2017

If I understand the code, the longer is the timeout, the slower is the frequency, no ? So you increased the frequency by 4 instead of lowing it, I think (without any tests).
Ping @damiencaselli.

@ghost
Copy link

ghost commented May 3, 2017

I think the error is in the commit message "set refresh at 25Hz", where the wanted unit was milliseconds. If not, it indeed does the opposite of what it is supposed to do.

@jjehl
Copy link
Author

jjehl commented May 4, 2017

Sorry, the right explanation is : increase the frequency from 10 Hz to 40 Hz.
At 10 Hz, motions are jerky.

@show0k
Copy link
Member

show0k commented May 11, 2017

I don't understand. If you increase refresh frequency, it will increase the load on the HTTP server and make the Raspberry Pi less CPU for motor control. So how increasing the frequency can make movements less jerky ?

@jjehl
Copy link
Author

jjehl commented May 11, 2017

This is a balance between the load of the server and the speed of the movement.
Low refresh frequency and high speed movement give jerky motion.

@show0k
Copy link
Member

show0k commented May 12, 2017

I really don't understand. Poppy monitor is used to visualize robot states, and start/stop primitives. In which way does it suppose to influence the frequency of movements started in the monitor ?

@jjehl
Copy link
Author

jjehl commented May 15, 2017

you can also move the motors with the mouse and this is for this feature that the refresh frequency is critical.

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