Satoshi Ikari

私もちゃんとレビューしたいので、マージはまだしないでください。

## 方針 - S2Eの中で有限要素法を解くなどはせず、外部の解析ツールから出力された構造パラメータをS2Eに入力して、S2E側ではダイナミクスを解くだけ - 構造パラメータの定義も整理していく - とりあえず片翼SAPで進めて良いが、最終的には両翼SAPに対応できるようにしたい ## 今後の流れ - @t-hosonuma から @fujiy へ、参考になる数式や参考文献の情報を渡す - @fujiy　の方でトルク形式の数式に変換する - 適宜、議論しながら - 問題なさそうなら実装に進む

@fujiy ありがとうございます！大きな方針は良いように思いますが、私もこの部分を専門で扱っているわけではないので、実際には実装して簡単なモデルで検証を行いつつ勧めていくことになるかと思います。 研究室の他のメンバーにも問題なさそうか確認してみます。

@conjikidow が興味を持っている。

遅くなりすみません。忙しいのが落ち着いたのでレビュー開始します。

@sksat @suzuki-toshihir0 こちら、だいぶ時間が経ってしまいましたが、どうしましょうか。ここまでで、次のどちらが良いかというのに議論は収束されたかなと思います。`initialize_files`というディレクトリを一度挟むかどうかという話です。 ``` - data - logs - initialize_files - simulation_base - environment - gnss.ini - gravity_field - geopotential, lunar gravity field - magnetic_field - igrf.coef - space_weather -...

AOCSミーティングで出た別の案は、dataディレクトリをなくして階層を浅くしたら良いのではというもの。srcなどと並列して、`logs`と`initialize_files`が並ぶ形 ``` - logs - initialize_files - simulation_base - environment - gnss.ini - gravity_field - geopotential, lunar gravity field - magnetic_field - igrf.coef - space_weather - SpaceWheather.txt - cspice -...

コメントありがとうございます。 > data/initialize_files にあったり ExtLibraries にあったり（さらにモノによっては src にあったり）して、非常に苦心していました............ これを解決するのが今回の提案なので、今回の提案以降では今までより明確に`シミュレーション設定のためのファイルが一つのディレクトリにまとまっている方が各種説明を行う時に楽`になると思っています。 > 新しい S2E user を用意した時ぐらいしかないと思っています 複数人でパラメータ（特定のセンサのノイズの大きさ、初期姿勢など）を色々振ってゲイン調整していくとき、少し条件を変えたものを共有するとき、昔試した条件を再現するため（logディレクトリに保存されたinitialize_filesをそのままコピーして再現する）などでコピーすることはたくさんあるかなと思います。 > ディレクトリにファイルが入っているのは自明なので _files という suffix は余計ではないでしょうか？ `initialize`が動詞で、他のディレクトリと同じように名詞にするために`files`がついているのだと理解しています。fileを外すなら、`initializer`など名詞にするのが良いのかなと思っています。 > data ディレクトリをなくす、というのはどのような観点での案なんでしょうか？ ディレクトリを深くしたくないからというのが理由です。私自身はあまり賛成はしていません。

> ディレクトリを深くしたくないの補足 ``` data - logs - initialize_files - いろんなinitializeファイル ``` が ``` data - logs - いろんなinitializeファイル ``` と比べて嫌な点は、`initialize_files`が一つあるせいで階層が深くなって、読みづらい　ということだったので、それならそもそも`data`がなければ一階層減らせるよという感じです。

議論ありがとうございます。議論の流れから、当初提案されていた「`initialize_files`的なディレクトリを削除する」という方向性はなくなったと思います。私も`initialize_files`的なディレクトリは必要と主張してきたので、賛成です。それは確定でお願いします。 > dataディレクトリが必要かどうか 上の流れから、@sksatさんが`階層を浅くするため、dataを消してsrcとlogsなどを並列にしたい`のか、`srcとlogsなどが並列なのは意味的に違和感なので、dataを維持して階層も今のままにする`のが良いのかどちらかいまいちわかりませんでした。 結局どちらでしょう？ 個人的には階層の浅さは特に重要でないので、`srcとlogsなどが並列なのは意味的に違和感なので、dataを維持して階層も今のままにする`が良いですが、強い意見ではないのでどちらでも良いとは思ってもいます。 > initialize_files の命名 元々は、`.ini`ファイルのことを指していましたが、上の提案の修正でそれ以外のファイルも入るのでこのタイミングで名前を変更するのはありだと思います。 `simulation_config`は略さないという命名のルール的には、`simulation_configuration`ですね。ただ、[このファイル名](https://github.com/ut-issl/s2e-core/blob/develop/src/simulation/simulation_configuration.hpp)と被るのが嫌だなと思います。 `settings`が個人的にはいいかなと思います。