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

BinaryデータでもASCIIデータでもないデータ形式にも対応したい #55

Closed
YusukeKato opened this issue Apr 10, 2024 · 1 comment
Labels
Type: Feature New Feature

Comments

@YusukeKato
Copy link
Contributor

現状の問題点

現在の実装だと9軸IMUのBinaryデータかASCIIデータ以外のデータを取得したときに、whileループが終わらない可能性がある。

該当箇所

while (rclcpp::ok() && driver_->hasCompletedFormatCheck() == false) {
  driver_->checkDataFormat();
}

https://github.com/rt-net/rt_usb_9axisimu_driver/blob/ros2-devel/src/rt_usb_9axisimu_driver_component.cpp#L96

原因:checkDataFormat()がBinaryかASCIIを判別するまでループするので、BinaryでもASCIIでもないデータを取得するとループが終わらない。

改善案

タイムアウト機能を追加する。

代替案

checkDataFormat()を実行する回数をカウントする。

その他

なし。

@YusukeKato YusukeKato added the Type: Feature New Feature label Apr 10, 2024
@YusukeKato YusukeKato changed the title 9軸IMUのBinaryデータかASCIIデータ以外のデータ取得に対応したい BinaryデータかASCIIデータ以外のデータ取得に対応したい Apr 10, 2024
@YusukeKato YusukeKato changed the title BinaryデータかASCIIデータ以外のデータ取得に対応したい BinaryデータでもASCIIデータでもないデータ形式にも対応したい Apr 10, 2024
@YusukeKato
Copy link
Contributor Author

#54 で解決したためクローズします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

No branches or pull requests

1 participant