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

Set stream freq to 80Hz #195

Merged
merged 1 commit into from
Jan 25, 2024
Merged

Set stream freq to 80Hz #195

merged 1 commit into from
Jan 25, 2024

Conversation

glannuzel
Copy link
Member

Setting stream freq to 80Hz seems to give enough time for the server to internally update its state and avoid overwriting of the goal_position

Setting stream freq to 80Hz seems to give enough time for the server to internally update its state and avoid overwriting of the goal_position
@glannuzel glannuzel self-assigned this Jan 24, 2024
@glannuzel glannuzel linked an issue Jan 24, 2024 that may be closed by this pull request
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1549 848 55% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/reachy2_sdk/reachy_sdk.py 41% 🟢
TOTAL 41% 🟢

updated for commit: 995260d by action🐍

Copy link
Contributor

@FabienDanieau FabienDanieau left a comment

Choose a reason for hiding this comment

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

All tests succeeded

@glannuzel glannuzel merged commit 7aeef85 into develop Jan 25, 2024
5 checks passed
@glannuzel glannuzel deleted the 191-fix-commands-problem branch January 25, 2024 10:50
This pull request was closed.
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.

Fix commands problem
2 participants