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

GS と FSW 側での同期のために,BCT, TL のダイジェスト (CRC) を下ろせるようにする App の追加 #268

Merged
merged 21 commits into from
Jan 5, 2024

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Dec 31, 2023

概要

GS と FSW 側での同期のために,BCT, TL のダイジェスト (CRC) を下ろせるようにする App の追加

Issue

検証結果

pytestを追加

影響範囲

  • これにともない,TL,BCTに保存される CCP の Sequence Count は 0 となるようにする

補足

手元検証用コマンド

# TL
# TL0 を空に
RT.MOBC.TLCD_CLEAR_ALL_TIMELINE 0
RT.MOBC.TL_BCT_DIGEST_TL 0 0
RT.MOBC.TG_GENERATE_RT_TLM 0x22

# TL 登録
TL.MOBC.NOP 100000
RT.MOBC.TL_BCT_DIGEST_TL 0 0
RT.MOBC.TG_GENERATE_RT_TLM 0x22
TL.MOBC.NOP 200000
RT.MOBC.TL_BCT_DIGEST_TL 0 0
RT.MOBC.TG_GENERATE_RT_TLM 0x22
TL.MOBC.NOP 300000
RT.MOBC.TL_BCT_DIGEST_TL 0 0
RT.MOBC.TG_GENERATE_RT_TLM 0x22

# BCT
# BCT を確認してクリア
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23
RT.MOBC.BCT_CLEAR_BLOCK 364
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23

# BCT 登録
BL.MOBC.NOP 1
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23
BL.MOBC.NOP 2
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23
BL.MOBC.NOP 3
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23

RT.MOBC.BCT_CLEAR_BLOCK 364
RT.MOBC.TL_BCT_DIGEST_BCT 364
RT.MOBC.TG_GENERATE_RT_TLM 0x23

@meltingrabbit meltingrabbit self-assigned this Dec 31, 2023
@meltingrabbit meltingrabbit changed the base branch from main to feature/ccp-dump-tlm December 31, 2023 07:12
@meltingrabbit meltingrabbit force-pushed the feature/tl_bct_digest branch 2 times, most recently from 2f7930b to dc7b384 Compare December 31, 2023 08:35
@meltingrabbit meltingrabbit changed the title WIP: GS と FSW 側での同期のために,BCT, TL のダイジェスト (CRC) を下ろせるようにする App の追加 GS と FSW 側での同期のために,BCT, TL のダイジェスト (CRC) を下ろせるようにする App の追加 Dec 31, 2023
@meltingrabbit
Copy link
Member Author

@KOBA789 FSW側はこんな感じになりそうです.
CSP の CRC 計算ですが,sequence count のトラッキングが厳しい & TL / BCT 登録後の sequence count には意味がないため,TL, BCT 登録時に,この count は 0 にするように変更しました.したがって, CRC は,送信した CSP のうち,sequence count を 0 にしたもので計算ください.

@meltingrabbit
Copy link
Member Author

@ToshiAki64 @hender14 レビューお願いします

Copy link
Collaborator

@ToshiAki64 ToshiAki64 left a comment

Choose a reason for hiding this comment

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

良さそうです。

Copy link
Contributor

@hender14 hender14 left a comment

Choose a reason for hiding this comment

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

コメントは入れましたが、処理内容は問題無いのでapproveする

@meltingrabbit meltingrabbit force-pushed the feature/tl_bct_digest branch from 6c6d0bb to 340a9ac Compare January 5, 2024 05:58
@meltingrabbit meltingrabbit merged commit bbef22b into main Jan 5, 2024
36 checks passed
@meltingrabbit meltingrabbit deleted the feature/tl_bct_digest branch January 5, 2024 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants