Skip to content

Conversation

@takasehideki
Copy link
Member

@takasehideki takasehideki commented Sep 16, 2023

#11 から fast-forward で進んでいます.

  • platform/mros2-platform.cpp は実はまだ使われていません.完全に実装しきれていません.
    • app_main() からこれの関数を呼び出して init_wifi()mros2::setIPAddrRTPS() したいつもりだったんですが,なぜか前者の時点でハングアップしてしまいます.C++リンカとの相性問題がありそうと思っています.いったん動いてるから:yoshi:
  • app_mainmros2::setIPAddrRTPS() に行くまでの道のりがひどいことは認識しています. ~~https://github.com/mROS-base/mros2-esp32/pull/13/files#diff-544503ce15d6276f13cffc3daede0b6e717c1c6aae3cdff578a77a5f8618b23aR41-R50 ~~
    • 型の cast なりでいい方法があれば教えてください.
    • というか mros2 側を改善したほうが早いかもしれません.
    • あるいは init_wifi() 内でやっちゃうのもありな気がしてきました.少なくともユーザからは隠蔽できるはず.

-> いずれも 4405aa7 で解決!setIPAddress を C module として実装して app_main から mros2_platform_network_connect() 呼び出したらよしなれるようにした.

@takasehideki
Copy link
Member Author

takasehideki commented Sep 17, 2023

@takasehideki
Copy link
Member Author

bc45793

新規のプロジェクトの common/git clone して使うときのユーザビリティを改善.

set(EXTRA_COMPONENT_DIRS ../common)

のママで編集せずに使えるようになったはず.

@takasehideki
Copy link
Member Author

takasehideki commented Sep 17, 2023

b8f3da6
sdkconfig 内の CONFIG_LWIP_IPV6 をビルド中に自動的に無効化できるようにした

@takasehideki
Copy link
Member Author

takasehideki commented Sep 17, 2023

  • platform/mros2-platform.cpp は実はまだ使われていません.完全に実装しきれていません.
    • app_main() からこれの関数を呼び出して init_wifi()mros2::setIPAddrRTPS() したいつもりだったんですが,なぜか前者の時点でハングアップしてしまいます.C++リンカとの相性問題がありそうと思っています.いったん動いてるから:yoshi:
  • app_mainmros2::setIPAddrRTPS() に行くまでの道のりがひどいことは認識しています. https://github.com/mROS-base/mros2-esp32/pull/13/files#diff-544503ce15d6276f13cffc3daede0b6e717c1c6aae3cdff578a77a5f8618b23aR41-R50
    • 型の cast なりでいい方法があれば教えてください.
    • というか mros2 側を改善したほうが早いかもしれません.
    • あるいは init_wifi() 内でやっちゃうのもありな気がしてきました.少なくともユーザからは隠蔽できるはず.

-> いずれも 4405aa7 で解決!setIPAddress を C module として実装して app_main から mros2_platform_network_connect() 呼び出したらよしなれるようにした.

@takasehideki
Copy link
Member Author

takasehideki commented Sep 17, 2023

mROS-base/mros2#51 に関する追従も行いました.
pub_twist を修正してあるので,動作確認用に試してみてください.

Copy link
Contributor

@s-hosoai s-hosoai left a comment

Choose a reason for hiding this comment

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

echoback_string, echoreply_string, pub_twistで動作確認取れました.特に回数で通信が詰まるような動作は見受けらえませんでした.
また単独のプロジェクトに導入し,app.cppを新しい方法に修正して動作確認できました.

@takasehideki takasehideki changed the title bump mros2 to v0.5.1 and v0.5.2, and follow these features bump mros2 to v0.5.1, v0.5.2, and vo.5.3!! and follow these features Sep 19, 2023
@takasehideki
Copy link
Member Author

takasehideki commented Sep 19, 2023

mros2 側を v0.5.3 でほぼ fix させたので,サンプルの動作確認を進めていきます.

  • echoback_string
  • echoreply_string
  • pub_float32
  • pub_twist
  • sub_pose
  • sub_uint16
  • pub_image (buildのみ)
  • pug_long_string_sub_crc (buildのみ)
  • m5stack_sample (buildのみ)

@takasehideki takasehideki changed the title bump mros2 to v0.5.1, v0.5.2, and vo.5.3!! and follow these features bump mros2 to v0.5.1, v0.5.2, and v0.5.3!! and follow these features Sep 19, 2023
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.

3 participants