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

Read the latest data regardless of the operation frequency #50

Closed
wants to merge 2 commits into from

Conversation

ShotaAk
Copy link
Collaborator

@ShotaAk ShotaAk commented Dec 13, 2023

What does this implement/fix?

#49 に関連します。

現在の実装では、IMUの最新データを受け取れるかどうかがtimerの動作周期に依存しています。
例えば、timerの周期を遅くすると最新データがほとんど取れなくなります。(遅れたデータを受信します)

publish_timer_ = create_wall_timer(10ms, std::bind(&Driver::on_publish_timer, this));

このPRでは、readを繰り返し実行し、常に最新データが取れるようにします。

Does this close any currently open issues?

#49

How has this been tested?

Binary、ASCIIモードで下記を確認しました。

  • timerの周期を100ms、5ms、10msに変更しても常に最新データが出力されること
  • on_configure時、USBが刺さっていないときにエラーを返すこと
  • on_activate時、USBが刺さっていないときにエラーを返すこと
  • activate中、USBを取り外すとエラーを返すこと

Any other comments?

Checklists

@ShotaAk ShotaAk added the Type: Bug Bug or Bug fixes label Dec 13, 2023
@YusukeKato YusukeKato mentioned this pull request Aug 21, 2024
2 tasks
@ShotaAk
Copy link
Collaborator Author

ShotaAk commented Aug 21, 2024

#58 で対応済みのためクローズします。

@ShotaAk ShotaAk closed this Aug 21, 2024
@ShotaAk ShotaAk deleted the fix_to_read_latest_value branch August 21, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug or Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant