-
Notifications
You must be signed in to change notification settings - Fork 18
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 earth albedo effect for thermal calculation #688
Conversation
アルベド計算をおこない,太陽光入熱とともに軌道上熱入力計算に反映するようにしました. |
@kai0722 ありがとうございます。
について,より詳細に数字やグラフなどが載せられたら載せてもらえると嬉しいです。 |
ログファイルでいいですかね? |
なるべくこちらがぱっと見て理解できるようにしてもらえると助かります 🙏 |
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.
修正案ありがとうございます。
コメントつけました。
また、コンフリクト解除もお願いします。
…ate related methods
アルベド計算式で,衛星が地球の影に入っているときはアルベド0になることを考慮できていなかったので,追加する. |
…or celestial positions
影判定はすでにLocaEnvironmentでやっているので、その機能を使ってください。 |
@kai0722 アルベドのiniファイルはその修正分もpushしてください。(ただし、計算のデフォルト値はDISABLEにしてください) |
… calculations and update method signatures
…adjust Earth albedo settings
User側でいじっていたため失念していました.反映しました. |
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.
修正ありがとうございます。議論も収束してきたのでこの指摘の修正が終わったらapproveできると思います。
…led to false in EarthAlbedo class
…lcEarthAlbedoEnabled methods
…grade-thermal-analysis
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.
コメントに対する小さな質問一つだけお願いします。CIが動いていないので、私の手元で動かして問題なければApproveします。
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.
CI治ったのでそのビルドの中でwarningが出ていたので修正をお願いします。
D:\a\s2e-core\s2e-core\src\dynamics\thermal\node.cpp(46,10): warning C4189: 'earth_distance_m': local variable is initialized but not referenced [D:\a\s2e-core\s2e-core\src\dynamics\DYNAMICS.vcxproj]
…/s2e-core into feature/upgrade-thermal-analysis
修正しました |
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.
修正ありがとうございます。Approveしたのでマージお願いします。
Related issues
#687
Description
The albedo calculation is now performed in the same way as for the calculation of solar heat input.
Test results
When not under eclipse, Albedo is reflected in the order of one-tenth of the solar heat input.Impact
The behavior of thermal analysis feature changes.
Supplementary information