概要

C89でビルドするためのSILS mockupと，github actionsを追加

Issue

• https://github.com/ut-issl/c2a-core/issues/20
• https://github.com/ut-issl/c2a-core/issues/27
• https://github.com/ut-issl/c2a-core/issues/2

詳細

https://github.com/ut-issl/c2a-core/issues/27 を参照

検証結果

actionsが通っていそうならOK

影響範囲

NA

補足

関連: https://github.com/ut-issl/c2a-core/pull/26

概要

Examples/minimun_user_for_s2eをビルドするだけ

Issue

• #20

詳細

とりあえずS2E with C2AではなくC2Aのみで完結したビルドCIが欲しい(C2A単体でのワーニングなどを認識するため)． そこで，exampleのsrc下にこのリポジトリをsrc/src_userとしてcloneしてしまい，ビルドする．

検証結果

windows runnerを使ってビルドが通った https://github.com/ut-issl/c2a-core/runs/4285174934?check_suite_focus=true

影響範囲

小: Examples/minimum_user_for_s2eのCMakeのオプションを追加しただけ．後でいいかんじにしたくはあるものの，とりあえず現状に即してはいる．

補足

いいかんじにやるならcoreのuser部への依存を整理したり色々やる必要があるが，とりあえず現状の状態でビルドCIを入れるならこう

概要

S2Eとリンクできるか確認するためのモックアップ

Issue

• #35 (Issueではないが)

詳細

C2A単体でビルドが通っていてもS2Eとリンクできないとつらい

検証結果

CIのWindows runnerでこれをビルドして通ればよし

影響範囲

N/A

概要

ビルドCI Actions の job nameが同じで，branch protectionで区別できないことの修正

Issue

• https://github.com/ut-issl/c2a-core/issues/30

詳細

https://github.com/ut-issl/c2a-core/issues/30 参照

検証結果

CIが通ればOK

概要

documentsを入れる

詳細

C2Aとは，各種tips，パラメタの説明，コーディングルールなどをいれる．

どこにどのように入れるかも議論する．

close条件

いれたら

概要

Fix #36

SILS専用のラッパー部分のC++としてしかコンパイルされない.cなファイルの拡張子を.cppにする．

Issue

• #36

詳細

N/A

検証結果

CIが通り，かつS2Eとリンクできればよし

影響範囲

S2Eが変更するファイル名に依存していればS2E側も修正する必要があるが，ヘッダを変えなければほぼ無いだろうと思われる．

補足

N/A

概要

ビルドCI Actions の job nameが同じで，branch protectionで区別できない

詳細

どっちも build_minimum_user になってるので．

https://github.com/ut-issl/c2a-core/blob/62e835382a108c6ec688c798d4b918e8ad28b598/.github/workflows/build_as_c89.yml#L18-L19

close条件

• [ ] 名前をユニークにしたら
• [ ] branch_protection_rules を再設定したら

概要

最低限S2Eでビルドする & 軽いチェックを行う CI を建てる

詳細

ビルドと規約チェックぐらい？

• [x] S2E ビルド
• [x] エンコーディングチェック
• [x] 規約チェック
• [ ] （できれば）テストCI

一旦HW依存ビルドは置いておく（coreなので）

https://github.com/ut-issl/c2a-core/issues/1 からの引き継ぎ内容

• 現在，S2EやWINGSの可搬性が高くないこともあるので，一旦ISSL内部でオンプレでもつ？
  • オンプレでもつ問題は https://github.com/ut-issl/c2a-core/issues/16
• user部をどうするか？ 簡素なsampleをつくってS2Eでまわすか？
  • その時，coreのPRでマージ前は，orgがut-isslではなく，フォーク先なので，user側のsubmoduleのURLを書き換える必要
  • → https://github.com/ut-issl/c2a-core/issues/2 で議論

close条件

CIがたったら

備考

NA

概要

src_user/IfWrapper/Sils/*.c にC++のコードを置くことについて議論する

詳細

• https://github.com/ut-issl/c2a-core/pull/35 でいろいろ話が出た
• これらはCとしてもC++としてもビルドされる，というわけではなく，C++としてしかビルドされない(SILS専用)
• それにも関わらず拡張子が.cになっており非常に分かりにくい
• さらに，CMakeでビルドする上でも都合が悪そう

close条件

なんとなく今後の方向性が見えてきたら．

概要

testをいれる

詳細

pytestで書かれたやつ．

どこにどのように入れるかも議論する．

close条件

いれたら

概要

c89のgcc Wextra Warningに対応すると，あるマイコンIDEにてWarningが出てしまう

詳細

• https://github.com/ut-issl/c2a-core/pull/73 にて， const uint32_t などの返り値を uint32_t などにした
  • warning: type qualifiers ignored on function return type [-Wignored-qualifiers] が発生
  • 基本型をconstつけて返しても，値渡しなため意味がないから
• 一方で，以下などでは，const構造体のメンバをconstなしで返している，として，あるマイコンIDEでは C0008 (W) Conversion in return を出してしまう．
  • https://github.com/ut-issl/c2a-core/blob/1280836a5c6a6a5fc7a4fc8a5a9211134345b5b5/Drivers/Super/driver_super.c#L1513-L1516

close条件

どちらの環境でもWarningがでなくなったら

備考

どうしようか．とりあえず，上の例だと，意味はないんだけど (uint16_t) とキャストさせる？

概要

Docs内部のリンクを修正

Issue

NA

概要

ELの軽微な性能改善

Issue

• https://github.com/ut-issl/c2a-core/issues/48

詳細

forを減らしたり，早く関数を返すようにしたり．

検証結果

既存のELのpytestが通ることを確認．

影響範囲

なし．

概要

• [x] pre-commit hook内で呼び出しているコマンドでエイリアスを実行しないようにする
• [ ] core.hooksPathを使うようにする(スクリプトで再帰しないようにする)

Issue

NA

詳細

以下で試みられていたが，\cはエスケープされるので\\cp ~にする必要があった https://github.com/ut-issl/c2a-core/blob/a6e93db543b2b210ba05f028233f5e008912c184/Script/Git/pre-commit.pl#L24

検証結果

test へのリンクや，検証結果へのリンク

影響範囲

cpやgitに対してaliasを設定していてもpre-commit hookでそれらが実行されなくなる

補足

何かあれば

概要

Cmd_BCT_OVERWRITE_CMD が可変長引数コマンドに対応できていない

詳細

param_len が可変長引数に対応できていない．

close条件

直ったら．

概要

コマンドのパラメタ取得の定型化

Issue

• https://github.com/ut-issl/c2a-core/issues/46

詳細

issueの要件をすべて満たす CCP_get_param_from_packet マクロを作成

雰囲気は，既存のコマンドの引数取得を修正したdiffを参照． Cmd_EL_RECORD_EVENT とかが劇的にきれいになっている．

検証結果

既存のtestを全部まわしてOKならOKでしょう

影響範囲

TBW

補足

関数ではなくマクロなのが若干気になる．

toolの更新はこれ

• https://github.com/ut-issl/c2a-tlm-cmd-code-generator/pull/2

あと，Cmd登録コマンドの修正と，testの追加をやる必要がある

概要

S2E側から呼び出すC2Aの初期化関数を，１つづつincludeして呼び出しているのはおかしい

詳細

void OBC_C2A::Initialize()

のなかに，各種initがいるのは良くなく，この中のinitはC2A側で持つべき．
C2A側でinit関数が変わったら，そのたびにS2E側も更新しないといけないのはおかしい．

close条件

きれいになったら

備考

• https://github.com/ut-issl/c2a-core/pull/67 ででた

概要

CoreとUserの分離，UserからCoreの利用のスキーマについて，改善を考える

詳細

2021年にそれまでOBCごとに１つのrepositoryで作業していたC2AをUserランドとCoreに分離し，すべてのOBCでCoreを共通化した．
一方で，あまり詳細を考えず，鈴本一人でがっと分離・実装してしまっているので，改善していきたい．

userとcoreでincludeが交錯しまくっているのとかもどうにかしたいし，パラメタ管理をもう少しいい感じにもしたい．

close条件

とりあえず議論から．議論してから考える

備考

https://github.com/ut-issl/c2a-core/issues/6 も関連しそう

概要

UTCなどの絶対時刻でTLC (Time Line Command) が打てるようにする

詳細

• OBC_TL.Cmd_HOGE において，時刻引数をtiではなく，utcなどの絶対時刻にて打てるようにする．
  • 既存の OBC_TL.Cmd_HOGE は残すので，あらたに追加することになる．
    • 命名は？ OBC_ATL (absolute tl)？，OBC_UTL (utc tl)？
• これは，あくまでエイリアス的な機能であることに注意する
  • 現在， OBC_TL.Cmd_HOGE は TCP_CMD_EXEC_TYPE_TL のTCP_EXEC_TYPEとなり，それはCCP_EXEC_TYPE_TL0のCCP_EXEC_TYPEに対応し，TL0に展開される．
  • ここで作るものは，コマンド受信部 　(https://github.com/ut-issl/c2a-core/blob/31dbe6e8c347252b55b6fc63ef7c11016b75e9b5/CmdTlm/packet_handler.c#L77) では，別のTCP_EXEC_TYPEとして受理されるが，最終的にはCCP_EXEC_TYPE_TL0に変換されるべき（つまり， https://github.com/ut-issl/c2a-core/blob/31dbe6e8c347252b55b6fc63ef7c11016b75e9b5/CmdTlm/packet_handler.c#L94 にルーティングされるべき）である．
  • つまり，引数はUTCなどであるが，それがPacketHandlerにて，TIへ変換され（これはObcTimeの関数で行う）た後に，TL0コマンドに変換され，TL0のキューに登録される．
• それに応じて，TCP_CMD_EXEC_TYPE はこのCmd用のtypeを追加する必要がある．
  • TCP_CMD_EXEC_TYPE などは 別途資料 を参照
• その他の注意
  • 新しい TCP_CMD_EXEC_TYPE の対応や，コマンドファイル対応などでWINGSの改修が必要．
    • これは早めにWINGSと調整すべき．C2Aの実装が終わる前にもWINGS側はリリースして問題ないはず．はやくリリースされたほうが，SILSなどでの検証が楽．
  • ObcTime (https://github.com/ut-issl/c2a-core/blob/31dbe6e8c347252b55b6fc63ef7c11016b75e9b5/System/TimeManager/obc_time.h) を一度整理してもいいかもしれない
    • 結構汚いコードが残ってるので
    • 型とか公開関数とか整理しても良いかも

close条件

実装ができたら

備考

• https://github.com/ut-issl/c2a-core/issues/63 の娘issue

概要

テレメ，コマンドの UnixTime への対応

詳細

以下 UTC ≒ UnixTime として書く

• MOBC の UTC 情報 を Tlm に入れる
• UTCを引数としてTLC登録を受理出来るようにする → https://github.com/ut-issl/c2a-core/issues/64
• TIのインクリメントと実時間の対応をGSから補正できるようにする

close条件

実装が終わったら

備考

NA