sbgisen / autoware_ai_planning

Apache License 2.0
0 stars 0 forks source link

Feature/noetic #34

Closed gakutasu closed 1 year ago

gakutasu commented 1 year ago

Summary

noeticはtfの名称に"/"を使用できないため、"/map"を"map"に修正。

Detail

Impact

Test

Attention

Tacha-S commented 1 year ago

melodic環境でのパラメータ調整作業ブランチから19272ecを追加してしまっていそうで、これらはまだ試行錯誤の段階(レビューされていない)ので、 noetic-devel(もしくはmaster)に19272ecのみコミットした形になるようにしてみましょう。

gakutasu commented 1 year ago

sbgisen/autoware_ai_planningにはもともとdevel-noeticブランチがなく、本日私が作成しました。 masterを親としてブランチを作ってしまったために、それまでのコミットがすべて反映されているのが本問題の原因だと考えています。

特定のコミット(私の変更分)のみマージする方法について、以下のような解決策を考えたのですが、坂口さんの認識とあっていますでしょうか。

feature/noeticはfeature/takeshibaを親としている これをmasterを親とするように変更(orブランチを作成)し、それをPRに出す

もしよりよい方法があれば教えていただきたいです。

(私としては現在調整中のパラメータも含めてマージしたいです、、)

Tacha-S commented 1 year ago

feature/noeticはfeature/takeshibaを親としている これをmasterを親とするように変更(orブランチを作成)し、それをPRに出す

これであっています。 履歴が変わるのでforce pushが必要になりますが、push先のブランチをfeature/noeticになるように気をつければ問題ないです。

(私としては現在調整中のパラメータも含めてマージしたいです、、)

これに関しては、その変更が何をどうやって解決しているのかを説明できる必要があります。 それに時間を割くぐらいならば @Alpaca-zip の作業がmelodic側でマージされ次第、 そのレビューに乗っかってnoeticにも同様の変更を取り込みますというPRを出すほうが楽だと思います。

gakutasu commented 1 year ago

仰るとおりだと思います。そのように対応したいと思います。

gakutasu commented 1 year ago

rebaseしてpush -fしました。 あっておりますでしょうか、ご確認お願いいたします。