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

Common Packet のデフォルト実装を core から提供する(Space Packet のみ) #329

Merged
merged 13 commits into from
Apr 8, 2024

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Apr 3, 2024

概要

  • 背景: CTCP, CTP, CCP のデフォルト実装を core で用意する #305
  • デフォルト実装として core が提供する SP の実装を使うかどうかを切り替えるビルドオプション C2A_USE_SPACE_PACKET_AS_COMMON_PACKET を追加している.
  • MOBCとsub OBCで実証が若干異なるため,それは C2A_MOBC_FEATURES オプションによってスイッチさせている
    • 現状は CTP_set_on_board_subnet_time のみ異なる
  • CTP の Global Time については,user ごとに異なる値をいれることができる
    • user 毎に src/src_user/settings/tlm_cmd/common_tlm_packet_define.c に定義をする必要がある.(CTP のうち,CTP_set_global_time のみこのファイルに分離した)
  • なお,文脈としては Common Packet のディレクトリ変更 #310 の続きである.

Issue

検証結果

CI がとおればOK

影響範囲

User に含まれる以下のファイルが不要になる

  common_cmd_packet.c
  common_tlm_cmd_packet.c
  common_tlm_packet.c

また, CTP_set_global_time を定義することが必須となり,

settings/tlm_cmd/common_tlm_packet_define.c

などを新規に作成する必要がある.
examples/subobc/src/src_user/settings/tlm_cmd/common_tlm_packet_define.c に例を配置した.

@meltingrabbit meltingrabbit self-assigned this Apr 3, 2024
@meltingrabbit meltingrabbit changed the title WIP: Common Packet のデフォルト実装を core から提供する(Space Packet のみ) Apr 3, 2024
@meltingrabbit meltingrabbit marked this pull request as ready for review April 3, 2024 08:57
@meltingrabbit meltingrabbit requested a review from sksat April 3, 2024 08:57
@meltingrabbit meltingrabbit added enhancement New feature or request priority::high priorityg high labels Apr 3, 2024
@meltingrabbit
Copy link
Member Author

@sksat さっき話した修正を施しました.レビューお願いします.

target_compile_definitions まわりで cmake のエラーでてるので,それも見てほしいです.

CMakeLists.txt Outdated Show resolved Hide resolved
@meltingrabbit meltingrabbit requested a review from sksat April 5, 2024 02:40
@meltingrabbit meltingrabbit force-pushed the feature/add_sp_defalt_impl branch from ebccc98 to 6261bc6 Compare April 5, 2024 09:46
@meltingrabbit meltingrabbit requested a review from sksat April 8, 2024 05:23
@meltingrabbit meltingrabbit merged commit 12c6b61 into main Apr 8, 2024
37 checks passed
@meltingrabbit meltingrabbit deleted the feature/add_sp_defalt_impl branch April 8, 2024 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::high priorityg high
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants