Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング #597

meltingrabbit · 2023-05-24T17:05:05Z

概要

Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング

Issue

詳細

cmdファイルと末尾のhow to use参照

検証結果

手元でテストした (末尾参照)
pytest つくりたいが，pytest の 5 Hz HK を壊すので悩み．．．

影響範囲

既存の TLM MGR が大幅アップデートかつ TG に対応

補足

Pre Release を打つ
マージする前にバージョンを上げる

→ やっぱやめ

Applications/telemetry_manager.h

meltingrabbit · 2023-05-25T07:41:05Z

@chutaro
ヘッダとcmd dbみてもらって，違和感ないかチェックもらってもいいですか？

実行時間がかなりでかいことが予想されるので，DCUを２箇所で使ってます．

.c と pytest は全然つくってないので，一旦無視で．

chutaro

とりあえず思ったことをコメントしています。詳細はcore mtg で相談できればと思います

Applications/telemetry_manager.h

Applications/telemetry_manager.c

Applications/telemetry_manager.h

Applications/telemetry_manager.c

meltingrabbit · 2023-05-29T07:31:58Z

@chutaro できた．
今なんとかtlmを1tlmに抑えようとテレメ圧縮してて，でき次第WINGSでテストします（このあと出張なので，今日の深夜頃を予定）

meltingrabbit · 2023-05-29T16:08:08Z

検証ログ

初期値

.# =======================================
.MOBC_RT.BCT_CLEAR_BLOCK 77     # BCT 77 を使用
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 1 0xf0 # HK
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 3 0x18 # TLM MGR 1
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 5 0x19 # TLM MGR 2
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 7 0x2a # DCU
 wait_sec 1
 MOBC_BL.TLCD_DEPLOY_BLOCK 10 2 77
 wait_sec 1
 MOBC_RT.BCE_ACTIVATE_BLOCK
 wait_sec 1
 MOBC_RT.TLCD_CLEAR_ALL_TIMELINE 2
 wait_sec 1
 # BCを展開し，TLM出力開始
 MOBC_RT.TLCD_DEPLOY_BLOCK 2 77
.#
.# =======================================
 #  TLM MGR
 # =======================================
.MOBC_RT.TLM_MGR_INIT

meltingrabbit · 2023-05-29T16:23:43Z

初期化

.# =======================================
 #  TLM MGR
 # =======================================
.MOBC_RT.TLM_MGR_INIT

BC ROLE や cmd type が OK そう

meltingrabbit · 2023-05-29T16:49:04Z

.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 0 0xf0   # HK, HK
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x18   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x19   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x20   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x21   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x25   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x26   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x2a   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x40   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x45   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x46   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x56   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x57   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x58   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x56   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x57   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x00   # LOW
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0xf1   # LOW
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x78   # LOW

登録OKそう

meltingrabbit · 2023-05-29T16:54:35Z

.MOBC_RT.TLM_MGR_START_TLM
.MOBC_RT.TLM_MGR_STOP_TLM
.MOBC_RT.TLM_MGR_START_TLM
.#

start, stop

meltingrabbit · 2023-05-29T17:20:07Z

.# 消す
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 1 0x20

.# 消す
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 1 0x20
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 1 0x56
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0xf1
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x78
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0xf0    # エラー
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x00
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x00    # エラー

複数 ID が登録されていても，ちゃんと１つだけ消える

meltingrabbit · 2023-05-29T17:20:26Z

最終的な検証コマンド

.# =======================================
.MOBC_RT.BCT_CLEAR_BLOCK 77     # BCT 77 を使用
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 1 0xf0 # HK
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 3 0x18 # TLM MGR 1
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 5 0x19 # TLM MGR 2
 wait_sec 1
 MOBC_BL.TG_GENERATE_MS_TLM 7 0x2a # DCU
 wait_sec 1
 MOBC_BL.TLCD_DEPLOY_BLOCK 10 2 77
 wait_sec 1
 MOBC_RT.BCE_ACTIVATE_BLOCK
 wait_sec 1
 MOBC_RT.TLCD_CLEAR_ALL_TIMELINE 2
 wait_sec 1
 # BCを展開し，TLM出力開始
 MOBC_RT.TLCD_DEPLOY_BLOCK 2 77
.#
.# =======================================
 #  TLM MGR
 # =======================================
.MOBC_RT.TLM_MGR_INIT
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 0 0xf0   # HK, HK
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x18   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x19   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x20   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x21   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x25   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x26   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x2a   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x40   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x45   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x46   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x56   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x57   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x58   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x56   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x57   # HIGH
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x00   # LOW
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0xf1   # LOW
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x78   # LOW
.#
.MOBC_RT.TLM_MGR_START_TLM
.MOBC_RT.TLM_MGR_STOP_TLM
.MOBC_RT.TLM_MGR_START_TLM
.#
.# 消す
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 1 0x20
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 1 0x56
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0xf1
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x78
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0xf0    # エラー
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x00
.MOBC_RT.TLM_MGR_DELETE_GENERATE_MS_TLM 2 0x00    # エラー
.#
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18   # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19   # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_CLEAR_HIGH_FREQ_TLM
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x18   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x19   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_CLEAR_LOW_FREQ_TLM
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x18   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 1 0x19   # HIGH, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18   # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19   # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 0 0x18    # HK, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 0 0x19    # HK, TLM MGR1
.MOBC_RT.TLM_MGR_CLEAR_USER_TLM
.MOBC_RT.TLM_MGR_CLEAR_HK_TLM
.#
.# 登録しすぎ
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 0 0xf0   # HK, HK
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x19    # LOW, TLM MGR1
.MOBC_RT.TLM_MGR_REGISTER_GENERATE_MS_TLM 2 0x18    # LOW, TLM MGR1 # err

MS tlm だけしか試してないが．．．

chutaro

ロジックまわりざっと見ましたが良さそうです！お疲れさまです。

変数名などでいくつかコメント入れてます。

Applications/telemetry_manager.h

Applications/telemetry_manager.c

chutaro · 2023-06-01T09:34:06Z

パス終了時にDR再生だけ1コマンドだけ止めたいという需要が結構あって、別プロではDR再生のコマンドを特定のBCに固めて、そのBCたちを inactivate することで対応していたのですが、座標がユーザーから見えなくなったのでこの実装だとできないですね。

別途DR側に、DR再生を無効化するコマンドを追加した方が良いかもです。

meltingrabbit · 2023-06-01T09:36:33Z

たしかに。
DRの元閉めるのは、DRにあれば良さそう。

chutaro · 2023-06-01T09:37:33Z

DCU使えば分割単位でPHに投げられるから、適切に分ければ3-3は出ないってことですよね？

chutaro · 2023-06-01T09:44:05Z

これはユーザー側でやれば良いと思いますが、sub OBC 数が少ないなら APID 1byteに圧縮して、

cmd type
apid
tlm id
dr partition

で4byte * 100 < 432 でコマンドテーブルだけは1テレメに抑えたいですね。

meltingrabbit · 2023-06-01T16:05:10Z

DCU使えば分割単位でPHに投げられるから、適切に分ければ3-3は出ないってことですよね？

そう

meltingrabbit · 2023-06-01T16:05:53Z

1テレメ

これ，APID下1桁で1byteにするとはいったんだけど，いまtlm dbが最大500行で，そっちに引っかかった．．．．．

meltingrabbit · 2023-06-01T16:43:43Z

@chutaro レビューありがとう！反映しました！

chutaro

お疲れさまです！！

chutaro · 2023-06-01T16:56:33Z

いまtlm dbが最大500行で，そっちに引っかかった

なるほど、、

meltingrabbit · 2023-06-01T16:59:36Z

#601

がすぐマージできるので，これリリース無しで行きます！

meltingrabbit added the priority::high priorityg high label May 24, 2023

meltingrabbit requested a review from chutaro May 24, 2023 17:05

meltingrabbit self-assigned this May 24, 2023

meltingrabbit changed the title ~~WIP: Feature/update tlm mgr for new tg~~ WIP: Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング May 25, 2023

github-actions bot reviewed May 25, 2023

View reviewed changes

Applications/telemetry_manager.h Outdated Show resolved Hide resolved

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch 2 times, most recently from 7e9faf6 to ccae225 Compare May 25, 2023 03:08

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch 2 times, most recently from f2133e8 to 4388fd3 Compare May 25, 2023 09:41

chutaro reviewed May 26, 2023

View reviewed changes

github-actions bot reviewed May 28, 2023

View reviewed changes

Applications/telemetry_manager.c Outdated Show resolved Hide resolved

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch 2 times, most recently from ce76d3a to e649bc7 Compare May 28, 2023 16:35

meltingrabbit mentioned this pull request May 29, 2023

TLM MGR の情報を元に，現在のビットレートを算出する App をつくる #598

Open

meltingrabbit changed the title ~~WIP: Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング~~ Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング May 29, 2023

meltingrabbit changed the title ~~Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング~~ Pre Release (v3.9.0-beta.8): Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング May 29, 2023

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch from ca4d7f4 to 3a5c75b Compare May 29, 2023 08:02

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch from d11a718 to 972e5e9 Compare May 29, 2023 17:24

meltingrabbit changed the title ~~Pre Release (v3.9.0-beta.8): Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング~~ Pre Release (v3.9.0-beta.9): Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング May 31, 2023

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch from 972e5e9 to c2a31d5 Compare May 31, 2023 08:34

sed bct_cmd_pos

dd86087

meltingrabbit added 5 commits May 31, 2023 18:23

update tlm

1146eb8

sed

154296c

fix bug

34bbed3

fix bug

e67606b

fix bug

dbc01a8

meltingrabbit force-pushed the feature/update_tlm_mgr_for_new_tg branch from c2a31d5 to dbc01a8 Compare May 31, 2023 09:23

meltingrabbit mentioned this pull request May 31, 2023

Pre Release (v3.9.0-beta.9): Mission Telemetry などの命名を Realtime Temeletry などにする #601

Merged

3 tasks

chutaro reviewed Jun 1, 2023

View reviewed changes

meltingrabbit added 5 commits June 2, 2023 01:32

fix typo

3508199

review

1583530

fix typo

4166111

sed TLM_MGR_CmdBlock

c372a8d

sed TLM_MGR_CmdElem

504929a

meltingrabbit requested a review from chutaro June 1, 2023 16:48

chutaro approved these changes Jun 1, 2023

View reviewed changes

meltingrabbit changed the title ~~Pre Release (v3.9.0-beta.9): Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング~~ Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング Jun 1, 2023

meltingrabbit merged commit f5a2571 into develop Jun 1, 2023

meltingrabbit deleted the feature/update_tlm_mgr_for_new_tg branch June 1, 2023 17:00

meltingrabbit mentioned this pull request Jun 1, 2023

Update main (v3.9.0) on 2023-06-02 #603

Merged

chutaro mentioned this pull request Aug 3, 2023

v3.10.0 に core update ut-issl/c2a-aobc#154

Merged

16 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング #597

Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング #597

meltingrabbit commented May 24, 2023 •

edited

Loading

meltingrabbit commented May 25, 2023 •

edited

Loading

chutaro left a comment

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023

chutaro left a comment

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

chutaro commented Jun 1, 2023

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

chutaro left a comment

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

Tlm Manager の 新 GENERATE_TLM 対応と，リファクタリング #597

Tlm Manager の 新 GENERATE_TLM 対応と，リファクタリング #597

Conversation

meltingrabbit commented May 24, 2023 • edited Loading

概要

Issue

詳細

検証結果

影響範囲

補足

meltingrabbit commented May 25, 2023 • edited Loading

chutaro left a comment

Choose a reason for hiding this comment

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 • edited Loading

検証ログ

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 • edited Loading

meltingrabbit commented May 29, 2023

meltingrabbit commented May 29, 2023 • edited Loading

meltingrabbit commented May 29, 2023

chutaro left a comment

Choose a reason for hiding this comment

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

chutaro commented Jun 1, 2023

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

chutaro left a comment

Choose a reason for hiding this comment

chutaro commented Jun 1, 2023

meltingrabbit commented Jun 1, 2023

Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング #597

Tlm Manager の新 GENERATE_TLM 対応と，リファクタリング #597

meltingrabbit commented May 24, 2023 •

edited

Loading

meltingrabbit commented May 25, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023 •

edited

Loading

meltingrabbit commented May 29, 2023 •

edited

Loading