-
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
Update v6.0.0 #303
Update v6.0.0 #303
Conversation
あ、ミスってこのブランチに直接pushしてしまった。 |
@suzuki-toshihir0 slackで議論したやつですが、b2c68fbと0f18abfで対応したので一応確認してもらい、変なところあったら教えて下さい。 |
変更の中身確認しました.問題ありません. |
…ializing-mode add attitude initialize mode
WIP: Small fixes before final merge for v6.0.0
Modify log output feature for Monte Carlo simulation
@suzuki-toshihir0 @seki-hiro @sksat I removed |
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.
Diffが非常に多いので,レビューの進め方は少し考えたいかもです.ほとんどのコードはすでに一度レビューが済んでいるのでだいたい良さそうであればapproveしてしまってよいのか,当時のレビュー抜けを拾うためにある程度丁寧に見る感じでいくべきか,想定あればお教えください.
基本は別PRでレビューしてくれているので、細かくみていく必要はないと思います。 |
わかりました.他のコードもまとめてざっと確認したしたが,大丈夫そうに思います. |
マージします。 |
Overview
Update v6.0.0
Issue
NA
Details
The following PRs are integrated into this PR
examples
directory #307components
directory #329GeoPotential
#359gen_graph
#361Simulation
directory to simplify for users #371simulation_sample
directory #372Validation results
See CI
Scope of influence
NA
Supplement
NA
Note
NA