-
Notifications
You must be signed in to change notification settings - Fork 19
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
Modify Calculation Function of Ground Image Position in Telescope #641
base: develop
Are you sure you want to change the base?
Modify Calculation Function of Ground Image Position in Telescope #641
Conversation
…dify_observe_ground_position_deviation_func
…dify_observe_ground_position_deviation_func
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] s2e-aobcの方にも似たコメントをつけましたが、こちら元々一般性の高い 本来はこのtelescopeをベースにTDI用のより特殊な |
…dify_observe_ground_position_deviation_func
@Hiro-0110 修正ありがとうございます。アルゴリズム面は問題ないと思います。 実装場所はここではなくなるということだったので、誤ってマージすることがないように、approveはしないでおきますね。 |
Related issues
Mention any issues that this pull request is related (e.g., #1). Consider using the
development
field if you want to close the issue automatically when this pull request is merged.Description
撮像時間幅のの開始時刻付近, 中央時刻付近, 終了時刻付近における, センサ中央に対応する地表点, センサy最大位置に対応する地表点, センサy最小位置に対応する地表点の動きを計算してグラフ化するようにした
Test results
Impact
Describe the scope of influence of the changes, e.g.,
The behavior of feature ** changes.
Supplementary information
Provide any supplementary information.