Skip to content

[upd] Arm#45

Open
HS-1114G wants to merge 7 commits intomainfrom
upd-arm
Open

[upd] Arm#45
HS-1114G wants to merge 7 commits intomainfrom
upd-arm

Conversation

@HS-1114G
Copy link
Copy Markdown
Collaborator

@HS-1114G HS-1114G commented Oct 8, 2021

変更の目的

・一通りの動作をラズパイ上で動くようにミスを編集した
・ヤコビ関連の部分でもしかしたらエラーが発生し、正しい値が出ていないかもしれない
・キム研のラズパイを使って来週もう一度試す予定


関連issue


レビュー完了希望日

2021/10/11 (月)

After merge

Fix #


・実際に動かし、エラーが発生した部分を修正
・ピグピオを使用した場合は、動作するはず
・サーボモータの動作
 →今まで:ぴぐぴお
 →このコード:PCA9685
・おおもとのフォルダにMainファイルを移動
・最後にテスト用に関数の呼び出し機能を付けた
・どこが実行されているのかをなるべくわかりやすくするために、デバッグ文を中心に記入を行った
Comment thread Arm/realsense.py
self.images = np.hstack((self.resized_color_image, self.depth_colormap))
else:
self.images = np.hstack((self.color_image, self.depth_colormap))
'''
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh !!! all English!!!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この部分はただ処理を軽くするために、端折る設定にしただけ
この部分を要約すると、カラー画像とデプス画像を左右に並べる配列にしますよ~ってこと。
カラー画像だけ見れればいいから、そんな無駄な処理をさせないようにコメントになっている。
※ワンちゃん使う可能性あるから完全には消してない

Comment thread Arm_main_pca9685.py
@@ -0,0 +1,172 @@
# 本プログラムの思考回路
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

日本語のときと英語のときの違いがようわからん

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

日本語
 →一括で消せる長文
 →長文を英語で書くのはだるいが、消すのは楽だから、しょうがなく日本語にしてる
英語
 →こまごまとした文章(3行程度ならなんちゃって英語で書くかも…)
 →ラズパイで編集しなくてもそのまま使えるため、英語で基本書いている

Comment thread Arm/arm_inverse.py
def moving(self):
# angles_trainsition (to return)
self.angles_trainsition = []
self.angles_transition = []
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここ何が変わったのかわからん

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trainsition = 訓練
transition = 遷移

@hs4449889
Copy link
Copy Markdown
Owner

pigpioからpca9685に変わった理由がよくわからなかったので説明指定だけませんか?

Copy link
Copy Markdown
Collaborator Author

@HS-1114G HS-1114G left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントしました。

Comment thread Arm/arm_inverse.py
def moving(self):
# angles_trainsition (to return)
self.angles_trainsition = []
self.angles_transition = []
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trainsition = 訓練
transition = 遷移

Comment thread Arm/realsense.py
self.images = np.hstack((self.resized_color_image, self.depth_colormap))
else:
self.images = np.hstack((self.color_image, self.depth_colormap))
'''
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この部分はただ処理を軽くするために、端折る設定にしただけ
この部分を要約すると、カラー画像とデプス画像を左右に並べる配列にしますよ~ってこと。
カラー画像だけ見れればいいから、そんな無駄な処理をさせないようにコメントになっている。
※ワンちゃん使う可能性あるから完全には消してない

Comment thread Arm_main_pca9685.py
@@ -0,0 +1,172 @@
# 本プログラムの思考回路
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

日本語
 →一括で消せる長文
 →長文を英語で書くのはだるいが、消すのは楽だから、しょうがなく日本語にしてる
英語
 →こまごまとした文章(3行程度ならなんちゃって英語で書くかも…)
 →ラズパイで編集しなくてもそのまま使えるため、英語で基本書いている

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