search

start-jsk / rtmros_tutorials

Tutorials for rtmros packages
6 stars 61 forks source link

Tablis #588

Closed ishiguroJSK closed 3 years ago

ishiguroJSK commented 3 years ago

TABLISのhrpsys_ros_bridge関連のファイル群です rtmlaunch hrpsys_ros_bridge_tutorials tablis.launch

Screenshot from 2021-02-04 19-49-16

副作用ないのでマージおねがいします

ishiguroJSK commented 3 years ago

ただのgitの使い方で申し訳ないのですが，git rebaseで過去のcommit内容修正すると，それ以降の他人のcommitも僕名義でごっそりrebaseされるわけですが，これで良いのでしょうか？ それとも，これやるくらいなら普通に追加で修正commitすればいいということになるんでしょうか？ Screenshot from 2021-02-04 22-58-45

pazeshun commented 3 years ago

ただのgitの使い方で申し訳ないのですが，git rebaseで過去のcommit内容修正すると，それ以降の他人のcommitも僕名義でごっそりrebaseされるわけですが，これで良いのでしょうか？

こうなる原因は、このbranchがorigin/masterに対してgit rebaseされておらず、origin/masterの変更をgit mergeで取り込んでいる状態で、origin/masterのcommitがこのbranch特有のcommitより後に来ているからです。 この場合、git rebaseでこのbranch特有のcommitを修正すると、origin/masterにmerge済みのcommitも変更されてしまい、このPRはconflict状態になる気がします。

おすすめの修正方法はgit rebase origin/masterする（commit logも綺麗になる）ことなのですが、branchの過去を変更したくない場合は、

追加で修正commitすればいい

となります。 一般的に、誰かのreviewが始まった後でrebaseすると、そのreviewのコメントが隠れてしまうため、git mergeと修正commitで対応することになりますが、今回はmerge commitがたくさんあって見にくいので、ぜひgit rebaseしてほしいです。

ishiguroJSK commented 3 years ago

git rebase -i origin/msater を開始して，僕のブランチ内に登場するorigin/masterとかからのmergeコミットを全部消しつつ，必要なcommitは修正しつつ，rebase作業を完了すれば良い？

pazeshun commented 3 years ago

まず、このPRのブランチにチェックアウトした状態で、 git rebase origin/master して(この時点でmerge commitは自動的に消失するはずです)、その後で、 git rebase -i (問題のあるcommitのうち最初のcommitの番号)~ して、各commitを修正してください。

ishiguroJSK commented 3 years ago

@pazeshun ありがとう！なんとか出来そうです．引き続き修正していきます

Naoki-Hiraoka commented 3 years ago

確認しました。

TABLIS.PDgains_sim.datの表記が現在のrtmros_choreonoidの仕様に対応していなかったため、修正したバーションのPRを作成しました。https://github.com/start-jsk/rtmros_tutorials/pull/595

それ以外の点は問題ないと思います。

Naoki-Hiraoka commented 3 years ago

URATA系は実機はRobotHardware.confを使うので，pdgains.file_nameはTABLIS.confではなくTABLIS.RobotHardware.confに書いてあるものが使われる，ということを見落としていました．(シミュレーションではTABLIS.confを使う)

2点修正しました．