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

MOBC 用 CCSDS のコード(主にデータリンク層)を core に移植する #214

Merged
merged 20 commits into from
Dec 3, 2023

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Nov 16, 2023

概要

GS と通信する SRX, STX が接続される OBC でしか使われないが,複数のOBCで使われるので,coreに移植する.
これによって,様々な MOBC で,CCSDS の AOS space data link protocol, TC space data link protocol 関連コードが core から提供可能になる.

Issue

詳細

NA

検証結果

CI が通ればOK

残作業

@meltingrabbit meltingrabbit added the enhancement New feature or request label Nov 16, 2023
@meltingrabbit
Copy link
Member Author

meltingrabbit commented Nov 21, 2023

sksatとはなした.以下案

  • ccsds/mobc ではなく ccsds/datalink_layer にする
  • ビルドオプションは C2A_USE_CCSDS_DATALINK_LAYER
  • で,既存の space packet のコードは ccsds/network_layer に移動する

参考

@sksat
Copy link
Member

sksat commented Nov 21, 2023

名前を付けるとしたら TC Space Data Link Protocol (なげえ)などの具体的なプロトコル名(レイヤーの名前で区切るのは無意味なので)

@meltingrabbit
Copy link
Member Author

@sksat フォルダ分け,プロトコル名でやってみて良い?

@sksat
Copy link
Member

sksat commented Nov 22, 2023

@meltingrabbit OK

CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
@meltingrabbit meltingrabbit changed the title [WIP] MOBC 用 CCSDS のコード(主にデータリンク層)を core に移植する MOBC 用 CCSDS のコード(主にデータリンク層)を core に移植する Nov 27, 2023
@meltingrabbit
Copy link
Member Author

@sksat @ToshiAki64

user setting をきりだす は,diffが見えなくなるので,別PRにします.
このPRでは以上にする予定ですので,レビューお願いします.

( @sksat changelog を追加済みなので,そこも見てほしい )

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@meltingrabbit meltingrabbit merged commit 941f294 into develop Dec 3, 2023
37 checks passed
@meltingrabbit meltingrabbit deleted the feature/move_ccsds_to_core branch December 3, 2023 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants