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

Add new branch and add the new velocity planning for high radio frequency. #149

Merged
merged 3 commits into from
Mar 4, 2024

Conversation

Fawo011
Copy link
Contributor

@Fawo011 Fawo011 commented Dec 31, 2023

No description provided.

Copy link

netlify bot commented Dec 31, 2023

Deploy Preview for rm-control-rm-controllers ready!

Name Link
🔨 Latest commit 6f1a152
🔍 Latest deploy log https://app.netlify.com/sites/rm-control-rm-controllers/deploys/65a26cb1b61fe60008920cfd
😎 Deploy Preview https://deploy-preview-149--rm-control-rm-controllers.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@d0h0s
Copy link
Contributor

d0h0s commented Dec 31, 2023

pr的标题首字母要大写;commit的标题首字母大写,加句号。

@Fawo011 Fawo011 changed the title add new branch and add the new velocity planning for low radio frequency Add new branch and add the new velocity planning for low radio frequency. Jan 5, 2024
@ye-luo-xi-tui
Copy link
Member

轨迹规划是要同时给出某个时刻的位置速度的,这样搞不如高频给个期望速度,低频不给。
如果想认认真真做一个规划器,建议再做一下调研,并把规划器封装成一个类。
据我所知低频的响应没啥问题吧,主要是高频时电机发热严重,那低频时不给期望速度估计也没啥问题

@Fawo011 Fawo011 changed the title Add new branch and add the new velocity planning for low radio frequency. Add new branch and add the new velocity planning for high radio frequency. Jan 13, 2024
@d0h0s
Copy link
Contributor

d0h0s commented Jan 17, 2024

测试情况怎样?

@Fawo011
Copy link
Contributor Author

Fawo011 commented Jan 17, 2024

测试情况怎样?

这个版本低射频是用的原先的纯位置规划,没什么大问题

@d0h0s d0h0s merged commit 6b89a28 into rm-controls:master Mar 4, 2024
6 checks passed
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.

3 participants