Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 8 commits
d5e4303
1dd6b06
aa91622
ed43b76
4dbbd53
ffab0ac
fb597bc
037b454
9f4d708
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
[Q] こちらはコードがほぼ変わっていないのに増えているのは何か想定外のことが起こっている気がします。
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.
HILSでは増えていました.HILSより実機か過去のデータを参照すべきですか?
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.
ここはHILSか実機かは関係ない部分なので、最新の実験結果を用いるというのが良いと思います。
cdh_update
の方に記入してくれている時間の総合計で30にしてくれているのだと思います。ここはAppCombinerなので、maxを全て足していくと過剰になってしまう可能性があります。なのでACでまとめた場合の時間を決めるのは難しく、ある程度の見積もりのもと3-3が出続けないラインまで短くしていったりしています。その差で7step分が生まれた可能性もありますが、元の23stepでずっと3-3アノマリが出続けるようでしたらやはりながくする必要はあるということになります。
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.
[Q] こちらもコードは変わっていないように思います。また、センサーデータの取得はHILSではなく実機で試験した方が良いと思いますが、実機でも3stepも伸びているのでしょうか?
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.
いえ,HILSで3step伸びています.本日書き込み,明日実機電気試験予定なのですが,明日の実機の電気試験よりも前に601で作業してapptimeを確認した方が良いでしょうか?
もしそうであれば,今からやるので,他にも実機のを採用すべきアプリがあれば教えていただけると助かります.
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.
DIなど実機センサと直接通信するものは、実機で測定するのが良いと思います。HILSの場合、HILS側の遅延が実機の遅延と一致していない可能性があるかなと思います。
特にI2Cは、DIの中でSlave側から返答があるまで待機します。そのSlave側からの返答スピードについて実機とHILSとで完璧に合わせれてはいないと思います。とはいえ、以前は実機側に合わせたらHILS側で3-3アノマリが出るなどはなく、「実機>HILS」だったのだと思います。
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.
[MUST] MTQ UPDATEは二つ入っているものの差が50になるように調整しています。
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.
変更します.
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.
ここも全て足すと17かもですが、ACなので過剰ということになっているかもしれません。それで6stepも差が生まれるのかなとは思いますが、元々の11stepで3-3アノマリが出続けるか見てみても良いかなとは思います。