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

Translate CMD DB to English #269

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

200km
Copy link
Member

@200km 200km commented Feb 1, 2024

Issue

NA

詳細

Translate TLM/CMD DB to English

検証結果

ビルドチェック (どちらもチェック)

  • SILSでのビルドチェックに通った(CIで確認)
  • vMicroでのビルドチェックに通った

動作確認チェック (いずれかをチェック)

  • SILSでアルゴリズムが想定通りに動いた
  • 実機でアルゴリズムが想定通りに動いた
  • (テレコマ試験の場合)コマンドファイルを使った試験をパスした

試験結果詳細記述場所 or 詳細ログ保存場所へのリンク

TBW

補足

The translations are supported by Chat GPT and DeepL.

@200km 200km added 📓 documentation Improvements or additions to documentation ✈️ priority::medium priority medium 🐟 patch Patch Update labels Feb 1, 2024
@200km 200km added this to the v9.0.0 Major Update milestone Feb 1, 2024
@200km 200km requested a review from SeirenTDoi February 1, 2024 17:55
@200km 200km self-assigned this Feb 1, 2024
@200km 200km requested review from sksat and a team as code owners February 1, 2024 17:55
@200km 200km requested review from suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team February 1, 2024 17:55
@200km 200km requested a review from seki-hiro February 8, 2024 14:48
@200km 200km changed the title [WIP] Translate TLM/CMD DB to English Translate CMD DB to English Feb 15, 2024
@200km
Copy link
Member Author

200km commented Feb 15, 2024

一旦CMD DBの翻訳をレビューしてもらえると嬉しいです。

@SeirenTDoi
Copy link
Collaborator

@200km 私の理解不足もあると思うのですが3点、コメントがございます。
・以下の中で、ANG_VELだけが日本語が完了で英語がanomalyとなっているのは意図通りでしょうか。
APP_AOCS_MM_SET_ANG_VEL_THRESHOLD
APP_AOCS_MM_SET_SUN_ANGLE_THRESHOLD
APP_AOCS_MM_SET_THREE_AXIS_THRESHOLD

・APP_SUN_POINTING_SAVE_GAIN等、5つのCMDで翻訳が「set by the above command」となっていますが、これはCMDの記述番に依存するので「set by the set command」等の方がいいようにも思うのですが、一方で記述順が現状のままなら現状の翻訳の方がわかりやすいと思いました。

・これは翻訳ではなく関数名がそうなのですが、DI_RW0003_DIAGNOSTICはRWからのReadと思いますが以下と異なりReadがついていないのは意味があるでしょうか。
DI_RW0003_READ_VDD
DI_RW0003_READ_SEU_COUNT
DI_RW0003_READ_FAULT_STATE

@200km
Copy link
Member Author

200km commented Feb 18, 2024

@SeirenTDoi ありがとうございます。

ANG_VELだけが日本語が完了で英語がanomaly

修正しました。

これはCMDの記述番に依存するので

二つで1セットのコマンドなので順番は基本変わらないですし、変わる修正が入った時にコメントも変えれば良いかなと思っています。

Readがついていないのは意味があるでしょうか。

おそらく意味はないですが、コマンド名自体を変えるのはmajor updateとなり、本PRの範疇外ですのでコマンド名の変更が必要でしたら別途issueを提案していただけると嬉しいです。

*レビュー機能でそれぞれのラインにコメントいただけると、わかりやすいと思うので、ご活用ください。

*,DR_SET_RP_SKIP_WIDTH_BY_TI,OBC,,2,uint8_t,Partition ID,uint32_t,period,,,,,,,,,,,Set read pointer skip width with TI,指定したTIおきにDLするSKIP WIDTH に設定
**,Memory commands,,,,,,,,,,,,,,,,,,,
*,MEM_SET_REGION,OBC,,2,uint32_t,Start address,uint32_t,End address,,,,,,,,,,,Set start address and end address,始点アドレスと終点アドレスを指定する
*,MEM_DUMP_REGION_SEQ,OBC,,2,uint8_t,category,uint8_t,Number of generation,,,,,,,,,,,Set categoly and send times,カテゴリと送信回数を指定する
Copy link
Member

Choose a reason for hiding this comment

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

スペルミス(同様のミスが他の箇所にもありました)

Suggested change
*,MEM_DUMP_REGION_SEQ,OBC,,2,uint8_t,category,uint8_t,Number of generation,,,,,,,,,,,Set categoly and send times,カテゴリと送信回数を指定する
*,MEM_DUMP_REGION_SEQ,OBC,,2,uint8_t,category,uint8_t,Number of generation,,,,,,,,,,,Set category and send times,カテゴリと送信回数を指定する

,NOP,OBC,0x0000,0,,,,,,,,,,,,,,,Dummy command just incrementing the command counter,コマンドカウンタを一つインクリメントするだけのダミーコマンド
,TMGR_SET_TIME,OBC,0x0001,1,uint32_t,TI,,,,,,,,,,,,,Set OBC time to TMGR(Time Manager),OBC時刻設定コマンド
*,TMGR_UPDATE_UNIXTIME,OBC,,3,double,Unixtime,uint32_t,Total cycle,uint32_t,Step,,,,,,,,,Modify OBC unixtime,OBC UNIXTIME修正コマンド
*,TMGR_SET_UTL_UNIXTIME_EPOCH,OBC,,1,double,UTL unixtime epoch,,,,,,,,,,,,,Set epoch unixtime usied in UTL_cmd,UTL_cmdで用いるunixtimeの紀元を変更する
Copy link
Member

@conjikidow conjikidow Mar 8, 2024

Choose a reason for hiding this comment

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

タイポ

Suggested change
*,TMGR_SET_UTL_UNIXTIME_EPOCH,OBC,,1,double,UTL unixtime epoch,,,,,,,,,,,,,Set epoch unixtime usied in UTL_cmd,UTL_cmdで用いるunixtimeの紀元を変更する
*,TMGR_SET_UTL_UNIXTIME_EPOCH,OBC,,1,double,UTL unixtime epoch,,,,,,,,,,,,,Set epoch unixtime used in UTL_cmd,UTL_cmdで用いるunixtimeの紀元を変更する

,BCE_TIMELIMIT_COMBINE_BLOCK,OBC,0x0023,2,uint16_t,Block command ID,uint8_t,Limit step,,,,,,,,,,,Timelimit combine block command,時間制限付きBLC融合コマンド
,BCE_RESET_ROTATOR_INFO,OBC,0x0024,0,,,,,,,,,,,,,danger,,Reset counter of rotator block command,カウンタ類リセット
,BCE_RESET_COMBINER_INFO,OBC,0x0025,0,,,,,,,,,,,,,danger,,Reset counter of combiner block command,カウンタ類リセット
,BCE_SET_ROTATE_INTERVAL,OBC,0x0026,2,uint16_t,Block command ID,uint16_t,Interval,,,,,,,,,,,Set rorate interval of rotater block command,BC回転時の回転間隔周期を設定する
Copy link
Member

Choose a reason for hiding this comment

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

タイポ・表現の修正

Suggested change
,BCE_SET_ROTATE_INTERVAL,OBC,0x0026,2,uint16_t,Block command ID,uint16_t,Interval,,,,,,,,,,,Set rorate interval of rotater block command,BC回転時の回転間隔周期を設定する
,BCE_SET_ROTATE_INTERVAL,OBC,0x0026,2,uint16_t,Block command ID,uint16_t,Interval,,,,,,,,,,,Set rotation interval of rotater block command,BC回転時の回転間隔周期を設定する

*,DR_SET_WRITE_ENABLE_FLAG,OBC,,2,uint8_t,Partition ID,uint8_t,flag,,,,,,,,,,,Set write enable flag,書き込み許可フラグ設定
*,DR_SET_RP,OBC,,2,uint8_t,Partition ID,uint32_t,pointer,,,,,,,,,,,Set read pointer,読み出しポインタ設定
*,DR_SET_RP_SKIP_WIDTH,OBC,,2,uint8_t,Partition ID,uint32_t,skip width,,,,,,,,,,,Set read pointer skip width,読み飛ばし幅設定
*,DR_REPLAY_TLM,OBC,,2,uint8_t,Category,uint8_t,Number of generation,,,,,,,,,,,Reply stored telemetry,蓄積パケット再生
Copy link
Member

Choose a reason for hiding this comment

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

スペルミス

Suggested change
*,DR_REPLAY_TLM,OBC,,2,uint8_t,Category,uint8_t,Number of generation,,,,,,,,,,,Reply stored telemetry,蓄積パケット再生
*,DR_REPLAY_TLM,OBC,,2,uint8_t,Category,uint8_t,Number of generation,,,,,,,,,,,Replay stored telemetry,蓄積パケット再生

,EL_CLEAR_STATISTICS,OBC,0x002A,0,,,,,,,,,,,,,,,Clear statistics information of the event logger,統計情報削除
,EL_CLEAR_TLOG,OBC,0x002B,1,uint8_t,Level: High=0; Low=1; EL=2; EH=3,,,,,,,,,,,,,Clear time seriese log (TLog) of the event logger,TLog削除
,EL_CLEAR_CLOG,OBC,0x002C,1,uint8_t,Level: High=0; Low=1; EL=2; EH=3,,,,,,,,,,,,,Clear cumulative log (Clog) of the event logger,CLog削除
,EL_RECORD_EVENT,OBC,0x002D,4,uint32_t,Group ID,uint32_t,Local ID,uint8_t,Level: High=0; Low=1; EL=2; EH=3,uint32_t,note,,,,,,,Recode event with the event logger,イベント記録
Copy link
Member

Choose a reason for hiding this comment

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

スペルミス

Suggested change
,EL_RECORD_EVENT,OBC,0x002D,4,uint32_t,Group ID,uint32_t,Local ID,uint8_t,Level: High=0; Low=1; EL=2; EH=3,uint32_t,note,,,,,,,Recode event with the event logger,イベント記録
,EL_RECORD_EVENT,OBC,0x002D,4,uint32_t,Group ID,uint32_t,Local ID,uint8_t,Level: High=0; Low=1; EL=2; EH=3,uint32_t,note,,,,,,,Record event with the event logger,イベント記録

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✈️ priority::medium priority medium 🐟 patch Patch Update 📓 documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants