-
Notifications
You must be signed in to change notification settings - Fork 4
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
HILSを用いたアプリタイムの調整 #307
HILSを用いたアプリタイムの調整 #307
Changes from all commits
d5e4303
1dd6b06
aa91622
ed43b76
4dbbd53
ffab0ac
fb597bc
037b454
9f4d708
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,11 +13,11 @@ | |
|
||
void BCL_load_inertial_ref_update() | ||
{ | ||
BCL_tool_register_app(0, AR_APP_TIME_SPACE_CALCULATOR); // 2step | ||
BCL_tool_register_app(1, AR_APP_ORBIT_CALCULATOR); // 2step | ||
BCL_tool_register_app(2, AR_APP_SUN_DIR_ECI_CALCULATOR); // 1step | ||
BCL_tool_register_app(3, AR_APP_GEOMAG_ECI_CALCULATOR); // 5step | ||
BCL_tool_register_app(4, AR_APP_TARGET_ATTITUDE_CALCULATOR); // 1step | ||
BCL_tool_register_app(0, AR_APP_TIME_SPACE_CALCULATOR); // 3step以上 | ||
BCL_tool_register_app(1, AR_APP_ORBIT_CALCULATOR); // 4step以上 | ||
BCL_tool_register_app(2, AR_APP_SUN_DIR_ECI_CALCULATOR); // 2step以上 | ||
BCL_tool_register_app(3, AR_APP_GEOMAG_ECI_CALCULATOR); // 7step以上 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ここのアプリが全体的に長くなっているのは気になりますね。TIME_SPACEもORBITもほとんど修正していない気がしています。 GEOMAGは一部の計算をdoubleにする修正が入っていますが、それだけで2stepも伸びてしまうのかなと思っています。これもHILS/実機関係ないところなので、最新の計測結果が正しいということになるとは思います。 |
||
BCL_tool_register_app(4, AR_APP_TARGET_ATTITUDE_CALCULATOR); // 1step以上 | ||
|
||
// total: 11step | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,26 +13,26 @@ | |
void BCL_load_bdot_mode(void) | ||
{ | ||
// CDH必須アプリ | ||
BCL_tool_register_combine(0, BC_AC_CDH_UPDATE); // 23step以上 | ||
BCL_tool_register_combine(0, BC_AC_CDH_UPDATE); // 30step以上 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [Q] こちらはコードがほぼ変わっていないのに増えているのは何か想定外のことが起こっている気がします。 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. HILSでは増えていました.HILSより実機か過去のデータを参照すべきですか? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ここはHILSか実機かは関係ない部分なので、最新の実験結果を用いるというのが良いと思います。
|
||
|
||
// AOCS必須アプリ | ||
BCL_tool_register_combine(25, BC_AC_BASIC_SENSOR_UPDATE); // 7step以上 | ||
BCL_tool_register_app (35, AR_APP_AOCS_MM_BDOT); // 1step以上 | ||
BCL_tool_register_combine(30, BC_AC_BASIC_SENSOR_UPDATE); // 10step以上 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [Q] こちらもコードは変わっていないように思います。また、センサーデータの取得はHILSではなく実機で試験した方が良いと思いますが、実機でも3stepも伸びているのでしょうか? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. いえ,HILSで3step伸びています.本日書き込み,明日実機電気試験予定なのですが,明日の実機の電気試験よりも前に601で作業してapptimeを確認した方が良いでしょうか? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. DIなど実機センサと直接通信するものは、実機で測定するのが良いと思います。HILSの場合、HILS側の遅延が実機の遅延と一致していない可能性があるかなと思います。 |
||
BCL_tool_register_app (45, AR_APP_AOCS_MM_BDOT); // 1step以上 | ||
|
||
// Bdotアプリ | ||
BCL_tool_register_combine(40, BC_AC_MTQ_UPDATE); // 1step以上, MTQ駆動時間カウンタを20 [Hz]周期にするため入れている | ||
BCL_tool_register_combine(50, BC_AC_MTQ_UPDATE); // 2step以上, MTQ駆動時間カウンタを20 [Hz]周期にするため入れている | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [MUST] MTQ UPDATEは二つ入っているものの差が50になるように調整しています。 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 変更します. |
||
|
||
BCL_tool_register_combine(45, BC_AC_RM3100_UPDATE); // 4step以上 | ||
BCL_tool_register_combine(55, BC_AC_RM3100_UPDATE); // 6step以上 | ||
|
||
BCL_tool_register_app (60, AR_APP_GYRO_SELECTOR); // 1step以上 | ||
BCL_tool_register_app (65, AR_APP_GYRO_SELECTOR); // 1step以上 | ||
|
||
|
||
BCL_tool_register_app (80, AR_APP_BDOT); // 1step以上 | ||
BCL_tool_register_app (70, AR_APP_BDOT); // 1step以上 | ||
|
||
BCL_tool_register_combine(90, BC_AC_MTQ_UPDATE); // 1step以上 | ||
BCL_tool_register_combine(75, BC_AC_MTQ_UPDATE); // 2step以上 | ||
|
||
// 基本使わないがユーザーが使いたい時に使えるように入れておく | ||
BCL_tool_register_combine(92, BC_AC_OEM7600_UPDATE); // 1step以上 | ||
BCL_tool_register_combine(92, BC_AC_OEM7600_UPDATE); // 3step以上 | ||
} | ||
|
||
#pragma section |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,32 +13,32 @@ | |
void BCL_load_fine_three_axis_mode(void) | ||
{ | ||
// CDH必須アプリ | ||
BCL_tool_register_combine(0, BC_AC_CDH_UPDATE); // 23step以上 | ||
BCL_tool_register_combine(0, BC_AC_CDH_UPDATE); // 30step以上(initial mode)→29step以上(fine three axis) | ||
|
||
// AOCS必須アプリ | ||
BCL_tool_register_combine(25, BC_AC_BASIC_SENSOR_UPDATE); // 7step以上 | ||
BCL_tool_register_app (35, AR_APP_AOCS_MM_FINE_THREE_AXIS); // 1step以上 | ||
BCL_tool_register_combine(29, BC_AC_BASIC_SENSOR_UPDATE); // 10step以上 | ||
BCL_tool_register_app (39, AR_APP_AOCS_MM_FINE_THREE_AXIS); // 1step以上 | ||
|
||
// 精三軸制御アプリ | ||
BCL_tool_register_combine(36, BC_AC_RM3100_UPDATE); // 4step以上 | ||
BCL_tool_register_combine(40, BC_AC_RM3100_UPDATE); // 6step以上 | ||
|
||
BCL_tool_register_combine(41, BC_AC_STIM210_UPDATE); // 1step以上 | ||
BCL_tool_register_combine(46, BC_AC_STIM210_UPDATE); // 3step以上→1step以上 | ||
|
||
BCL_tool_register_combine(45, BC_AC_MTQ_UPDATE); // 1step以上, MTQ駆動時間カウンタを20 [Hz]周期にするため入れている | ||
BCL_tool_register_combine(47, BC_AC_MTQ_UPDATE); // 2step以上, MTQ駆動時間カウンタを20 [Hz]周期にするため入れている→1step以上 | ||
|
||
BCL_tool_register_combine(47, BC_AC_SUN_VECTOR_UPDATE); // 合計4step以上必要ある? | ||
BCL_tool_register_combine(51, BC_AC_INERTIAL_REF_UPDATE); // 11step以上 | ||
BCL_tool_register_combine(48, BC_AC_SUN_VECTOR_UPDATE); // 6step以上 | ||
BCL_tool_register_combine(54, BC_AC_INERTIAL_REF_UPDATE); // 17step以上(initial mode)→16step以上(fine three axis) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ここも全て足すと17かもですが、ACなので過剰ということになっているかもしれません。それで6stepも差が生まれるのかなとは思いますが、元々の11stepで3-3アノマリが出続けるか見てみても良いかなとは思います。 |
||
|
||
BCL_tool_register_combine(63, BC_AC_STT_UPDATE); // 4step以上 | ||
BCL_tool_register_app (68, AR_APP_FINE_THREE_AXIS_DETERMINATION); // 1step以上 | ||
BCL_tool_register_combine(70, BC_AC_STT_UPDATE); // 13step以上 | ||
BCL_tool_register_app (83, AR_APP_FINE_THREE_AXIS_DETERMINATION); // 2step以上 | ||
|
||
BCL_tool_register_combine(75, BC_AC_OEM7600_UPDATE); // 1step以上 | ||
BCL_tool_register_combine(85, BC_AC_OEM7600_UPDATE); // 3step以上→1step以上 | ||
|
||
BCL_tool_register_app (80, AR_APP_THREE_AXIS_CONTROL_RW); // 1step以上 | ||
BCL_tool_register_app (84, AR_APP_UNLOADING); // 1step以上 | ||
BCL_tool_register_app (86, AR_APP_THREE_AXIS_CONTROL_RW); // 1step以上 | ||
BCL_tool_register_app (87, AR_APP_UNLOADING); // 1step以上 | ||
|
||
BCL_tool_register_combine(85, BC_AC_RW_UPDATE); // 10step以上 | ||
BCL_tool_register_combine(95, BC_AC_MTQ_UPDATE); // 1step以上 | ||
BCL_tool_register_combine(88, BC_AC_RW_UPDATE); // 10step以上 | ||
BCL_tool_register_combine(98, BC_AC_MTQ_UPDATE); // 2step以上→1step以上 | ||
} | ||
|
||
#pragma section |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[NITS] コメントアウトのインデントを揃えてもらえると綺麗かなと思います。 (他も同様です)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あと、これまではこちらでメモとっていたことは把握していますかね?下記はオープンじゃないので直接コードに書いても良いかもですが、一応お伝えしておきます。
https://docs.google.com/spreadsheets/d/13fAz0aFXkwjRiu-oOnvo85XZ5n9_quoh8s3XFROh1HY/edit#gid=0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
存じていなかったです.