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

ブランチ整理 #25

Open
MoriKen254 opened this issue Jul 22, 2020 · 2 comments
Open

ブランチ整理 #25

MoriKen254 opened this issue Jul 22, 2020 · 2 comments

Comments

@MoriKen254
Copy link
Member

From https://github.com/Nishida-Lab/denso_recognition/pull/88

@ShunjiroOsada

こちらの branch network を確認し、要否について考察コメントを、よろしくお願いします。

@YamafukuKei
Copy link
Contributor

calm0815 の2つのブランチに関しては,master と比較したところ,変更点はないようです.そのため,この2つのブランチは消してしまっても問題ないと思います.

yokota/update-firmware についてですが,これは photoneo ファームウェアのアップデートかつ,phoxi_camera リポジトリをFork していたものから,公式の最新のものに切り替えたことによる仕様変更に対応して,機能を実装したブランチになります.以前は,古いファームウェアをまだ使う必要があるかもしれない,CI が通らない(phoxi_camera が必要とするソースのインストールで詰まったため)といった問題があり,マージされない状態になっていました.

そして,yamafuku/add_get_one_srv についてですが,これはロボット動作中に phoxi_camera から点群を絶えず取得していると,ネットワーク帯域を大きく占有してしまい,結果的にロボットの緊急停止を起こしやすくなるという理由から,任意のタイミングで phoxi_camera を起動して,センサ情報を取得するためのサービスを実装したものになります.このブランチにあるノードを denso_recognition の honda/matching_test ブランチでは必要としていました.

結論としては,以下のようになります.

  • calm0815 の2つのブランチは消してよい
  • yamafuku/add_get_one_srv については,honda/matching_test でも使用されたが,仕様に問題なければ PR どおりに yokota/update-firmware にマージ
  • yokota/update-firmware については,これから新しい photoneo のファームウェアを使用する方針なので,CI の問題を解決するか,別 issue とするかによりますが,master にマージする方針

@MoriKen254
Copy link
Member Author

ありがとうございます。

  • calm0815 のブランチを削除しました。
  • yamafuku/add_get_one_srv は、レビューの上マージしましょう。
  • yokota/update-firmware は、古いファームウェアを使う可能性を捨てきれない間は、残しておきましょう。

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

No branches or pull requests

2 participants