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

TlmCmdDBフォーマットv3対応 #20

Open
KOBA789 opened this issue Dec 6, 2023 · 3 comments
Open

TlmCmdDBフォーマットv3対応 #20

KOBA789 opened this issue Dec 6, 2023 · 3 comments
Assignees

Comments

@KOBA789
Copy link
Member

KOBA789 commented Dec 6, 2023

ref: #26

  • v3 の CSV をパースできるようにする
    • v2(現状)の CSV もパースできる状態を保つ
  • v3 の CSV から tlmcmddb.json を生成できるようにする
  • v3 の tlmcmddb.json を読み込めるようにする
    • v2 の tlmcmddb.json も読み込める状態を保つ

仕様はこのとおり: ut-issl/tlm-cmd-db#24

@kobkaz
Copy link
Contributor

kobkaz commented Dec 6, 2023

V3で書かれたcsvデータってどこかにありますか?
@KOBA789

@kobkaz
Copy link
Contributor

kobkaz commented Dec 6, 2023

V2とV3の相違点をここにまとめる

syntax上の違い

  • 1行1列にバージョン情報が入る
  • TLM EntryTLM Field に修正
  • Display Info.Label Unit format カラムの追加
  • DesiginatorDesignator に修正
  • %%## (\r\n) -> ## (\n)
    • 現行ではチェックを飛ばしているので気にしない

semantics上の違い

semanticsの検査はparserとは別にvalidatorがあるほうがよろしい

  • TLM FieldName.で区切られた階層構造になっており、あるNameが別のNameの階層のprefixになっていることは許されない
    • X.Y と X.Y.Z は共存不可 X.YとX.YY は共存可?

@KOBA789
Copy link
Member Author

KOBA789 commented Dec 6, 2023

v3 で書かれた CSV はこの PR に含まれてます!(Issue description に貼ってあるやつと同じ PR)
ut-issl/tlm-cmd-db#24

sksat pushed a commit that referenced this issue Dec 11, 2023
Cmd DB から "Cmd_" prefix を削除
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants