Satoshi Ikari
Satoshi Ikari
ご返信ありがとうございます。では、次のような感じで修正しようと思います。 ``` - src - logs - settings - simulation_base - environment - gnss.ini - gravity_field - geopotential, lunar gravity field - magnetic_field - igrf.coef - space_weather - SpaceWheather.txt -...
#613 に引き継いで作業も始めているので、これはCLOSEします。
@shunichironomura 原因がわかっているわけではないですが、M1 Macというのが若干引っかかっています。 下記の記事など参考になりますかね? https://github.com/docker/for-mac/issues/6030 https://pythonspeed.com/articles/docker-build-problems-mac/
解決したようで良かったですが、この修正を反映すべきか考えたいのでreopenします。 dockerfileを上手く書けばM1 Macでもそうでなくても動くようにできるのであればそうしたいです。 最低限、documentの手順に注釈で入れたいかなと思っています。
@ItsukiKawamoto コメントありがとうございます!M1 Macユーザーの方に確認をお願いしていますので、少々お待ち下さい。
@ItsukiKawamoto 解決したようで良かったです! @shunichironomura コメントありがとうございます!!
修正提案ありがとうございます。ただ、Major Update v8.0.0に関するPRが溜まっており、レビューが進んでいない状況です。 こちらはissueにも立てられておらず、優先度は下げて良いという認識ですので、v8.1.0のマイルストーンに回し他を優先するということでよいでしょうか。
ライブラリディレクトリが整理されたので新しい構成に合わせて名前空間も見直す。 ディレクトリ構成に合わせるなら、次のようになるか? とりあえず、今存在している`libra`に焦点を当てて考える。`library`ディレクトリがほぼそのまま`math_physics`に変わったので、次のような感じか? ``` c++ namespace math_physics{ namespace math{ Class Vector; Class Matrix; 他も同様 } namespace atmosphere{ Class HarrisPriester; Class Nrlmsise00; } } ``` もしくは、 ``` c++ namespace math_physics{ Class...
コメントありがとうございます。私も下記のものは良いと思っているので、この方向で進めようと思います。 ```c++ namespace math{ Class Vector; Class Matrix; 他も同様 } namespace atmosphere{ Class HarrisPriester; Class Nrlmsise00; } ```
@conjikidow コメントありがとうございます。とりあえず今出ているPR消化してから改めて整理して考えたいのでレビューお願いします。