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

STTの大きいテレメ追加 #170

Merged
merged 2 commits into from
Aug 14, 2023
Merged

Conversation

seki-hiro
Copy link
Member

@seki-hiro seki-hiro commented Aug 14, 2023

Issue

#159

詳細

バッファサイズが大きくなったので、コメントアウトしておいたテレメを追加する。
主な変更

  • TLM DB更新
  • SAGITTA_RX_MAX_FRAME_SIZEを350に増やす
  • テレメフレームが大きくなったことに伴う引数の型変更 (uint8_t -> uint16_t)

検証結果

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

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

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

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

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

CENTROIDS

ESCAPEなしで168Byte受信し、フレーム確定できている。

c0 21 84 25    00 f0 d0 d4    ed ce 97 1b    4a 01 5d 03    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 14    51 27 0a c0
DSSC_get_rec_status(stream_config)->status_code: 5

MATCHED CENTROIDS

ESCAPEなしで343Byte受信し、フレーム確定できている。

c0 21 84 28    00 1b f1 b1    d7 ce 3f 48    3e 01 5d 03    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 ab 41    9f f8 c0
DSSC_get_rec_status(stream_config)->status_code: 5

HISTOGRAM

ESCAPEなしで166Byte受信し、フレーム確定できている。

c0 21 84 1c    00 04 d4 c8    25 f2 79 8a    4d 01 5d 03    00 01 00 00    00 00 00 00    00 00 00 00    00 ce 24 00    00 9b 03 0e    00 95 d7 01    00 01 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 00 00 00    00 01 00 00    00 de 09 00    00 c6 02 0e    00 59 f3 01    00 02 00 00    00 00 00 00    00 00 00 00    00 07 00 00    00 01 00 00    00 04 00 00    00 f2 09 00    00 b0 2b 0d    00 48 ca 02    00 0a 00 00    00 00 00 00    00 00 00 00    00 0a 00 00    00 01 00 00    00 07 00 00    00 60 02 00    00 da 44 0d    00 ae b8 02    00 06 00 00    00 00 00 00    00 00 00 00    00 24 17 de    a9 c0
DSSC_get_rec_status(stream_config)->status_code: 5
スクリーンショット 2023-08-14 14 24 07

CENTROIDSやMATCHED CENTROIDSは、WINGS UI上で0のままだが、初期値を変えるときちんと更新されていることを部分的に確認済み。

補足

NA

@seki-hiro seki-hiro added this to the 開発仮目標1 milestone Aug 14, 2023
@seki-hiro seki-hiro self-assigned this Aug 14, 2023
@seki-hiro seki-hiro requested review from sksat and a team as code owners August 14, 2023 01:30
@seki-hiro seki-hiro requested review from 200km, suzuki-toshihir0 and conjikidow and removed request for a team August 14, 2023 01:30
@seki-hiro
Copy link
Member Author

@conjikidow STTテレメ受信できました㊗️

Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

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

問題ないです。

@seki-hiro seki-hiro merged commit 53f2b81 into develop Aug 14, 2023
@seki-hiro seki-hiro deleted the feature/add_stt_large_tlm branch August 14, 2023 07:41
@seki-hiro seki-hiro mentioned this pull request Aug 14, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

バッファサイズ変更後にSagittaのテレメ表示を増やす
2 participants