Ryo Suzumoto

Results 142 comments of Ryo Suzumoto

> Tlm headerをすべて手打ちできる低レベルコマンドはつくる > Destination Flags を打たなくていい便利コマンドを作る? 書き方悪かったね.`Dest Flags` と `DR Partition` です. これ直打ちするモチベは,flagを複数あげて,DR記録しながらMS tlmとして下ろす,とかができるって感じ. とはいえ,ほぼ使わない(使わず済むならつかいたくない)って感じなので,別途 dest flagごとのgenerate tlmコマンドほしいなぁって(そっちのほうがミスが少ない & コマンドファイル見たときに意図がわかりやすい)思ったんだけど,なくてもいいか. そしたら `Cmd_TG_GENERATE_TLM ${tlm id} ${dest_flags} ${dr_partiton} ${送出回数}` って感じかしら?

> 既存の Cmd_GENERATE_TLM は後方互換性のために残す これは残しましょうか

> MOBCだけ特別で、他は各コンポにコンポID割り振ってGENERATE_COMPO_TLM で良い気がします、というかコンポIDではなくAPIDで良い? "コンポ" ってのが不明瞭な単語で,space pacektを出力しないコンポは,APIDはMOBCになるし,MOBCの普通のテレメと一緒なので普通に(新しく作るコマンドの方の) `Cmd_GENERATE_TLM` で大丈夫. 一方,space pacektを出力するAOBCとかは,パケットをMOBCでためていて,それを配送するためのコマンドが必要って意味かな.この場合,tlm idはMOBCとは独立なので,別途コマンドが必要. って意味で, `COMPO_TLM` はちょっとミスリードだなあってのと, `GENERATE` してなくね? っていうのも悩みポイント.

DRとの親和性考えて(DRからの再送,と別OBC packetのbufferからの再送って意味で) ``` Cmd_TG_REPLAY_TLM ${obc指定enum or APID} ${tlm id} ${dest_flags} ${dr_partiton} ${送出回数} ``` ってのは? APID,複雑だしAPIDから特定のOBCを引っ張ってくるの今の構造だと大変なので(Cだとdict型がないので...),enumかなぁ...と

> Enum作ってsubOBC叩くのとAPID経由でsubOBC叩くのって変わらなくないですか? うん,変わんない. enumだと,配列にできて,sub obcのtlm packet buffer叩くところをいい感じにできるなって思ったのと,APID長いなぁ...って思っただけなんだけど,新しいindex作るのはそれはそれで管理がめんどくさいし,今後の拡張考えたらAPIDにすべきなので,APIDにしますか. > これで良いと思いますが、個人的にはちょっと性質が変わるのでMOBCのDRと subOBCからのbufferTLMは分けたい気がします(結局内部でDR叩くか、buffer 叩くかで変わってきて、内部関数変わるので) 分ける予定よ.(一緒にするつもり無いよ) DRはどのOBC packet入ってるかわからないし.(=DRはAPIDをしらない) DRは引き続き, ``` Cmd_DR_REPLAY_TLM ``` の予定

tlm managerとの兼ね合いがちょっとめんどくさいなこれ. `Cmd_TG_REPLAY_TLM` でAPIDが自分だったら `Cmd_GENERATE_TLM` に読み替えてほしいけど,それはそれでミスリーディングっていう...

``` CCP_EXEC_STS Cmd_TLM_MGR_REGISTER_HK_TLM_FOR_GENERATE_TLM(const CommonCmdPacket* packet); CCP_EXEC_STS Cmd_TLM_MGR_REGISTER_HK_TLM_FOR_REPPLAY_TLM(const CommonCmdPacket* packet); CCP_EXEC_STS Cmd_TLM_MGR_REGISTER_HK_TLM_FOR_REPLAY_DATA_RECORDER(const CommonCmdPacket* packet); ``` みたいにするしかないのか?

- https://github.com/ut-issl/c2a-core/issues/155#issuecomment-1027098869 はこっちで引き取る.

`#error` って別に `""`で囲わなくてもいいので,これは規約チェックスクリプトが良くないっぽいな

あーたしかに.... 今度直します mm (とはいえ,これはほぼ買えないので軽微か)