8000 [Stabilizer.cpp] fix typo of https://github.com/fkanehiro/hrpsys-base… by eisoku9618 · Pull Request #922 · 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.

Sign up for GitHub

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

Jump to bottom

[Stabilizer.cpp] fix typo of https://github.com/fkanehiro/hrpsys-base… #922

Merged
merged 1 commit into from
Dec 23, 2015
Merged

[Stabilizer.cpp] fix typo of https://github.com/fkanehiro/hrpsys-base… #922

merged 1 commit into from
Dec 23, 2015

Conversation

eisoku9618
Copy link
Contributor

…/pull/895

#895 のときの消し忘れでしょうか？ > @snozawa さん

上の変更のときに数行上で同じ回転行列をかけているので，二回かけてしまっているように見えます．

@k-okada
Copy link
Contributor
k-okada commented Dec 22, 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/2499/
Test PASSed.

@snozawa
Copy link
Contributor
snozawa commented Dec 22, 2015

すいません、そのようです。

@eisoku9618
Copy link
Contributor Author

こちら，ロボットの挙動としては何も変わらず，ログに保存される値が正しくなる，という変更です．

マージしていただけると幸いです．
よろしくお願いします．

fkanehiro added a commit that referenced this pull request Dec 23, 2015
@fkanehiro
Merge pull request #922 from eisoku9618/patch-1
05e5eba 
[Stabilizer.cpp] fix typo of https://github.com/fkanehiro/hrpsys-base…
@fkanehiro fkanehiro merged commit 05e5eba into fkanehiro:master Dec 23, 2015
@eisoku9618 eisoku9618 deleted the patch-1 branch December 23, 2015 09:40
@eisoku9618
Copy link
Contributor Author

ありがとうございました．

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewer 4E98 s
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@eisoku9618 @k-okada @snozawa @fkanehiro
0