ut-issl / s2e-core

Spacecraft Simulation Environment Core codes
MIT License
47 stars 19 forks source link

namespaceの追加 #701

Closed 200km closed 1 month ago

200km commented 2 months ago

Related issues

326

Description

全ての要素に対して、namespaceを追加した。結果、次のような名前空間となった。 (ファイル一つ一つ見てレビューは大変だと思うので、下記が問題ないと思うならApproveして欲しいです)

- componentsディレクトリ内:s2e::components
- disturbancesディレクトリ内:s2e::disturbances
- dynamicsディレクトリ内
  - attitudeディレクトリ内:s2e::dynamics::attitude
  - orbitディレクトリ内:s2e::dynamics::orbit
  - thermalディレクトリ内:s2e::dynamics::thermal
- environmentディレクトリ内:s2e::environment
- loggerディレクトリ内:s2e::logger
- math_physics内:s2e::をつけて、その下はこれまで通り
- simulationディレクトリ内
  - ground_stationディレクトリ内:s2e::ground_station
  - spacecraftディレクトリ内:s2e::spacecraft
  - それ以外:s2e::simulation
- simulation_sampleディレクトリ内:s2e:sample
- utilitiesディレクトリ内:s2e::utilities

Test results

CI確認

Impact

NA

Supplementary information

NA

github-actions[bot] commented 2 months ago

position.png

position

github-actions[bot] commented 2 months ago

position.png

position

github-actions[bot] commented 2 months ago

position.png

position

200km commented 1 month ago

ご指摘いただいた点は迷いましたが、ディレクトリ分けの方が変だなという気もしています。(かといってspacecraftディレクトリを一つ上の階層にそのまま持っていくのも変かなと思っています) math_physics/hogeにも全てmath_physics::つけているわけではないので、そこと同じかなとは思っています。

200km commented 1 month ago

とりあえずこの巨大PRは一旦マージして細かく議論した方が良いと思うので、マージします。