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

APID enum の命名規則を変えたい #519

Closed
Tracked by #616 ...
meltingrabbit opened this issue Mar 16, 2023 · 3 comments
Closed
Tracked by #616 ...

APID enum の命名規則を変えたい #519

meltingrabbit opened this issue Mar 16, 2023 · 3 comments
Labels
priority::high priorityg high

Comments

@meltingrabbit
Copy link
Collaborator

meltingrabbit commented Mar 16, 2023

概要

APID enum の命名規則を変えたい

詳細

typedef enum
{
APID_MOBC_CMD = 0x210, //!< 01000010000b: APID for MOBC 宛の CMD
APID_AOBC_CMD = 0x211, //!< 01000010001b: APID for AOBC 宛の CMD
APID_TOBC_CMD = 0x212, //!< 01000010010b: APID for TOBC 宛の CMD
APID_TCAL_TLM = 0x410, //!< 10000010000b: APID for TIME CARIBLATION TLM (FIXME: 現在まともに使ってない)
APID_MOBC_TLM = 0x510, //!< 10100010000b: APID for MOBC で生成される TLM
APID_AOBC_TLM = 0x511, //!< 10100010001b: APID for AOBC で生成される TLM
APID_TOBC_TLM = 0x512, //!< 10100010002b: APID for TOBC で生成される TLM
APID_DUMP_TLM = 0x710, //!< 11100010000b: APID for DUMP TLM (FIXME: 現在まともに使ってない)
APID_FILL_PKT = 0x7ff, //!< 11111111111b: APID for FILL PACKET
APID_UNKNOWN

これ

現在,TLM,CMDともに,受信先,送信元がGSであることを暗に仮定しているためそうなっているが,今後,MOBC発コンポA向け,といったものも出てきて,FROM,TOを識別したくなってくる.

そうすると,CMD,TLMは頭につけたくなってくる?

たとえば

APID_CMD_TO_MOBC     (APID_CMD_TO_MOBC_FROM_GS の短縮)
APID_CMD_TO_A_FROM_B
APID_TLM_FROM_AOBC     (APID_CMD_TO_GS_FROM_AOBC の短縮)
APID_TLM_TO_B_FROM_A

close条件

考えたら

関連

@meltingrabbit meltingrabbit added the priority::high priorityg high label Mar 16, 2023
@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Mar 16, 2023

パスが複数ある場合にどう対応するか

APID_TLM_FROM_A_TO_B_VIA_C

とか

あとは,SバンドのGSは省略して,Xで落とすときは

APID_TLM_FROM_A_TO_XGS (TO_SGS,FROM_SGS は省略できる,という整理)
APID_TLM_FROM_A_VIA_X (TO_GS を省略し,SのときはVIA_Sを省略しているという意味)

とか?

@meltingrabbit
Copy link
Collaborator Author

C2A Core MTG の議論

Sバンドがデフォルトで使ってる環境ならSGSを省略できるとして,
APID_TLM_FROM_A_TO_XGS (TO_SGS,FROM_SGS は省略できる,という整理)
とするのがいいのでは?

@meltingrabbit
Copy link
Collaborator Author

arkedge/c2a-core#186 でやってくので閉じ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high priorityg high
Projects
Status: Done
Development

No branches or pull requests

1 participant