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

テレコマDBのパスを変えて標準化する #142

Closed
Tracked by #28
sksat opened this issue Oct 12, 2023 · 3 comments · Fixed by #172
Closed
Tracked by #28

テレコマDBのパスを変えて標準化する #142

sksat opened this issue Oct 12, 2023 · 3 comments · Fixed by #172
Assignees
Labels
enhancement New feature or request priority::medium priority medium tools
Milestone

Comments

@sksat
Copy link
Member

sksat commented Oct 12, 2023

現在,テレコマDBのパスは以下のようになっている.

  • C2A user
    • src
      • src_user
        • Settings
          • tlm_cmd
            • data_base
              • CMDFILE
              • CMD_DB
              • TLM_DB
                • calced_data

パッと見てわかるように,これは階層が深すぎる.

テレコマDBは衛星毎・対向 OBC 毎に用意し,C2A のコード生成に用いられるため,たしかに単にいち C2A user だけから見ると設定の一種のように見えるが,これは他の OBC(FSW)・他の C2A user との間のインターフェースを記述するためのスキーマであり,システム全体からの視点で C2A user のテレコマDBを見ると,これはいち C2A user そのものが C2A user の外に対して提供する API であり,単なる設定ではない.
また,実務上でも,C2A user のコードそのものを開発しない人でも,テレコマDBを閲覧・編集することは多い.

そのため,テレコマDBが非常に深い階層にあるのは不適切であると考えられる.

@sksat sksat added enhancement New feature or request priority::high priorityg high tools labels Oct 12, 2023
@sksat sksat self-assigned this Oct 12, 2023
@sksat
Copy link
Member Author

sksat commented Oct 12, 2023

現在のバージョンの tlm-cmd-code-generator ではこのパスは可変になっている: ut-issl/c2a-tlm-cmd-code-generator#45

そのため,この変更は直近の major update(今回の場合は v4.0.0)ではなく,新たに標準的なテレコマDB置き場のパスという概念を作り,移行期間を設ける.

@sksat
Copy link
Member Author

sksat commented Oct 12, 2023

最近の ISSL は src と同じディレクトリに database というディレクトリを作っているので,一旦これに合わせるのがツールの互換性などからもよさそう.

@sksat sksat added priority::medium priority medium and removed priority::high priorityg high labels Oct 12, 2023
@sksat
Copy link
Member Author

sksat commented Oct 26, 2023

単に example user でのディレクトリを変えるだけならやるだけだし,ISSL と歩調を合わせる意味でも v4 でやっておく.v4.0.0 ではあくまで推奨の場所とする(つまり,移行期間は設ける)

@sksat sksat added this to the v4.0.0 milestone Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::medium priority medium tools
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant