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 force generator #18

Merged
merged 10 commits into from
Jul 25, 2022
Merged

Add force generator #18

merged 10 commits into from
Jul 25, 2022

Conversation

200km
Copy link
Member

@200km 200km commented Jul 24, 2022

概要

Add force generator

Issue

#3

詳細

Add force generator

検証結果

  • A result of following settings
    • true ordered force at body frame : [1, 0, 0] N
    • magnitude error standard deviation: 0.01 N
    • direction error standard deviation: 1 deg

Fig1. The generated force norm (horizontal: time [sec], vertical: force [N])

  • The magnitude error successfully added.

image


Fig2. The distribution of generated force direction in YZ plane (horizontal: body Y axis [deg], vertical: body Z axis [deg])

  • The direction error successfully added.

image

影響範囲

Small

補足

NA

注意

NA

@200km 200km added the priority::high priorityg high label Jul 24, 2022
@200km 200km requested a review from suzuki-toshihir0 July 24, 2022 21:31
@200km 200km self-assigned this Jul 24, 2022
@suzuki-toshihir0
Copy link
Member

WIPですが,レビュー始めてしまっても大丈夫でしょうか?

Copy link
Member

@suzuki-toshihir0 suzuki-toshihir0 left a comment

Choose a reason for hiding this comment

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

要修正点は特になさそうです.approveします.

@200km 200km changed the title WIP: Add force generator Add force generator Jul 25, 2022
@200km 200km requested a review from suzuki-toshihir0 July 25, 2022 13:29
@suzuki-toshihir0
Copy link
Member

ノイズ部についても問題なさそうです!

@200km 200km merged commit 48675e7 into develop Jul 25, 2022
@200km 200km deleted the feature/add-force-generator branch July 30, 2022 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high priorityg high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants