Skip to content

[hrpsys_choreonoid/iob.cpp] load pdgains.file_name on initialize#354

Merged
YoheiKakiuchi merged 1 commit intostart-jsk:masterfrom
Naoki-Hiraoka:PR-initialgain
Jun 10, 2021
Merged

[hrpsys_choreonoid/iob.cpp] load pdgains.file_name on initialize#354
YoheiKakiuchi merged 1 commit intostart-jsk:masterfrom
Naoki-Hiraoka:PR-initialgain

Conversation

@Naoki-Hiraoka
Copy link
Contributor

#349 の一部です。

実機はservo offの状態から起動しますが、シミュレーションでは時間短縮のためにいきなり立った状態から開始するので、実機ではservo on時にhrpsysで行われる処理を、シミュレーションではiob内で最初から行う必要があります。

pdgains.file_nameの値を実機に入力する処理は、実機ではservo on時(厳密にはwrite_pgain等が呼ばれたとき)にhrpsysで行われます。このPRによって、シミュレーションでは開始時にiob内でpdgains.file_nameの値を読むようになります。

関連するissue #326 (comment)

この変更によって、実機とシミュレーションで同じpdgains.file_nameのファイルを使用することが可能になります。rtmros_choreonoid_tutorials以下にあるロボットの中で、実機とシミュレーションで異なるpdgains.file_nameのファイルを使っているロボットについて、start-jsk/rtmros_tutorials#597 で修正しました。

@Naoki-Hiraoka
Copy link
Contributor Author

@YoheiKakiuchi
テストが通りました. よろしければmergeをお願いできますでしょうか.

@YoheiKakiuchi YoheiKakiuchi merged commit c17c26d into start-jsk:master Jun 10, 2021
@Naoki-Hiraoka Naoki-Hiraoka deleted the PR-initialgain branch June 10, 2021 06:31
@Naoki-Hiraoka
Copy link
Contributor Author

ありがとうございます.

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.

2 participants