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

[DO NOT MERGE] Tlm DB v3 #24

Closed
wants to merge 7 commits into from
Closed

[DO NOT MERGE] Tlm DB v3 #24

wants to merge 7 commits into from

Conversation

meltingrabbit
Copy link
Collaborator

@meltingrabbit meltingrabbit commented Jul 13, 2023

概要

Tlm DB v3

Issue

詳細

詳しく

検証結果

これから core 側の 改修いれます.先んじてレビューお願いできますか?

影響範囲

v2

image

v3

image

補足

何かあれば

@meltingrabbit meltingrabbit changed the title Tlm DB v3 [DO NOT MERGE] Tlm DB v3 Jul 13, 2023
@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jul 13, 2023

もともとは, C2A core v4 と v3 系で,同じ tlm cmd db を使いたかったため,v4 fork の前に,この改修をいれたかった.
しかしながら,現状の仕様は,後方互換性の観点から,サポートを切れないので,ここでの仕様策定を急ぐ理由はなく,v4はv4で,tlm cmd dbを考えていく方針とした.

つまり,一旦マージを凍結する.

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Nov 14, 2023

  • 「TLM Entry」はタイポなのでtlm fieldに(csv)
  • 例をきちんと入れる
  • 「.」は階層を区切ると明記する
    • これは禁止(ディレクトリとファイル名で名前をかぶらせない、と同じ)
    • HOGE.FUGAHOGE.FUGA.PIYO の両立禁止
  • ここに tlm xlsm が tlm db のバージョンを焼き込むようにする
    • image

@@ -1,38 +1,68 @@
# TlmCmd DB
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

スキーマのある tlm と、スキーマのない tlm のそれぞれの名前をきめたい

blob tlm: memdumpなど.可変長.また,分割あり
struct tlm: tlm db で定義されるもの

@meltingrabbit
Copy link
Collaborator Author

以後は arkedge/c2a-tlmcmddb#26 で実施する

@meltingrabbit
Copy link
Collaborator Author

arkedge/c2a-tlmcmddb#72 で実施

meltingrabbit added a commit to arkedge/c2a-tlmcmddb that referenced this pull request Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant