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
Add ObserveGroundPositionDeviation Function #529
Add ObserveGroundPositionDeviation Function #529
Changes from 3 commits
1e0fe65
e8813e2
09709c8
e0adaed
42fb3f7
eff64e2
d0abb32
6b6d06e
dd2a9bb
d3b53f6
496e040
d9b27ad
afffafe
3998cb2
e6bc2af
bd4796c
2f32834
b23b148
2d9bde0
517b33e
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.
ECEF座標系での衛星の座標をもとめて(initial_spacecraft_position_ecef_m_)、それに対して
地球半径を掛けて、地球半径と高度の和で割る事によってECEF座標系での直下点の座標(initial_ground_position_ecef_m_)を求めています。
ベクトルをスカラーで割ろうとするとエラーになるのでやり方がわからず、紛らわしい書き方になっています、、
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.
やりたいこと、なんとなくわかりました。
こんな感じでも同じ意図になりますかね?(理解のために尋ねているだけなので、こう修正しなくても良いです。)
こちらに関しては、現状は次のようにコーディングする必要があります。
今回の場合は、
みたいな感じですかね。