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

Update dbus communication protocol. #210

Closed
wants to merge 3 commits into from

Conversation

YoujianWu
Copy link
Contributor

在DbusData.msg里加了rc_is_open值,用于判断遥控是否打开,并更新dbus相应逻辑。

Copy link

netlify bot commented Mar 31, 2024

Deploy Preview for rm-control ready!

Name Link
🔨 Latest commit 4cd1c82
🔍 Latest deploy log https://app.netlify.com/sites/rm-control/deploys/6608fb838c0e950008beba5c
😎 Deploy Preview https://deploy-preview-210--rm-control.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.

@ye-luo-xi-tui
Copy link
Member

这样改的好处是什么,为什么不像以往一样,其他节点自己判断dbus有没有开

@YoujianWu
Copy link
Contributor Author

YoujianWu commented Apr 4, 2024

这样改的好处是什么,为什么不像以往一样,其他节点自己判断dbus有没有开

由于ethercat2dbus在更新数据时,时间戳一直在刷新,时间戳是与硬件同步的,无法直接通过判断时间戳来判断dbus开关,manual无法兼容usb转串口和ethercat转dbus。为了兼容两套硬件,所以作了相关改变。

@qiayuanl
Copy link
Member

qiayuanl commented Apr 8, 2024

这样改的好处是什么,为什么不像以往一样,其他节点自己判断dbus有没有开

由于ethercat2dbus在更新数据时,时间戳一直在刷新,时间戳是与硬件同步的,无法直接通过判断时间戳来判断dbus开关,manual无法兼容usb转串口和ethercat转dbus。为了兼容两套硬件,所以作了相关改变。

You should use a bit in the statusword to indicate whether the dbus is on/off.

@qiayuanl qiayuanl closed this Apr 8, 2024
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