8000 Use average of force sensor values for force sensor offsetting and use it in removeforcesensorlinkoffset by snozawa · Pull Request #455 · fkanehiro/hrpsys-base · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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

Use average of force sensor values for force sensor offsetting and use it in removeforcesensorlinkoffset #455

Merged
merged 1 commit into from
Jan 28, 2015

Conversation

snozawa
Copy link
Contributor
@snozawa snozawa commented Jan 28, 2015

力センサのゼロ点だしのremoveForceSensorOffsetで、時間平均をとるようにしました。
方法はcalibrateInertiaSensorと同じです。

IDLレベルのAPIはかわりません。
実際に行われる処理は仕様変更となりますが、
時間平均をとっているだけですのでremoveForceSensorOffsetを特殊な使い方をしてないかぎりは
悪影響がないと考えて折ります。
問題なさそうでしょうか。

よろしくお願いいたしあmす。

@snozawa
Copy link
Contributor Author
snozawa commented Jan 28, 2015

時間平均をとっているだけですのでremoveForceSensorOffsetを特殊な使い方をしてないかぎりは

時間平均の他にisBusyのif文などエラーチェックは追加されてることになりますが、ベーシックな機能の変更としては時間平均を追加、のみになると思います。

@k-okada
Copy link
< 954D span aria-label="This user has previously committed to the hrpsys-base repository." data-view-component="true" class="tooltipped tooltipped-n"> Contributor
k-okada commented Jan 28, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1733/
Test PASSed.

fkanehiro added a commit that referenced this pull request Jan 28, 2015
Use average of force sensor values for force sensor offsetting and use it in removeforcesensorlinkoffset
@fkanehiro fkanehiro merged commit 2983194 into fkanehiro:master Jan 28, 2015
@snozawa snozawa deleted the use_average_foffset branch January 28, 2015 17:30
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
0