Skip to content

Conversation

@AyumuIwa
Copy link

No description provided.

@AyumuIwa
Copy link
Author

AyumuIwa commented Nov 30, 2022

@sktometometo
launchファイルを作成しカメラ画像から試してみたのですが、位置が上手く表示されていないように思います。どうしたらよろしいでしょうか?
よろしくお願いします。

20221130_panorama2dto3d.mp4

<node
pkg="nodelet"
type="nodelet"
name="panorama_image_resize"
Copy link
Contributor

@sktometometo sktometometo Nov 30, 2022

Choose a reason for hiding this comment

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

panorama_image をリサイズするとpanorama_info も変更する必要があるため、 => 必要ない
DualFisheyeToPanorama の方で出力解像度を変更することをおすすめします
https://github.com/jsk-ros-pkg/jsk_recognition/blob/e8e5ac3ef91ed416682ff33bba04407b33e83af8/jsk_perception/sample/sample_insta360_air.launch#L27

Copy link
Contributor

Choose a reason for hiding this comment

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

これにはCPU使用率が無駄にならない利点も有ります

<remap from="~input_class" to="$(arg INPUT_CLASS)" />
<remap from="~input_rects" to="$(arg INPUT_RECTS)" />
<rosparam>
frame_fixed: "camera"
Copy link
Contributor

Choose a reason for hiding this comment

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

frame_fixed は名前の通り 世界座標に固定された座標系で使うことを意図しているので、 odom とかにしておくことをおすすめします。

</include>

<include file="$(find coral_usb)/launch/edgetpu_panorama_object_detector.launch">
<arg name="INPUT_IMAGE" value="/dual_fisheye_to_panorama/output"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

rect_array_in_panorama_to_bounding_box_array に入力しているのが quater の方でこちらは quater でないので合わせて下さい。

@sktometometo
Copy link
Contributor

上記を直して試してもらえると助かります

@AyumuIwa
Copy link
Author

AyumuIwa commented Dec 1, 2022

@sktometometo

ご対応ありがとうございます。試してみます。

@knorth55
Copy link
Member

knorth55 commented Dec 2, 2022

@AyumuIwa こんな感じでlaunchを修正してみましたがどうでしょうか?
AyumuIwa#1

@AyumuIwa
Copy link
Author

AyumuIwa commented Dec 2, 2022

ありがとうございます。
すみませんが、明日試してみます。

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.

4 participants