-
Notifications
You must be signed in to change notification settings - Fork 17
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
Create data/external dir & move GeoPotential files #612
base: develop
Are you sure you want to change the base?
Conversation
修正提案ありがとうございます。 今の状況ですが、
という感じです。なので、 今の分類の方針としては、
という感じになっていると思います。 方針を変更して整理する場合、次のような特性を考慮して分け方を考えた方が良いのかなと思います。
この特性を踏まえて、 また、ここに出てきておらず、みてみぬふりをしているものとして、
|
はい.これはその通りで,あくまで最初のミニマルな改善提案だというのと,単にレビューコストを下げるためにパッチを分割していただけです > Geopotentialだけ動かすのはそれはそれで整合性が取れなくなる 必須データかどうか,によって分け方を考えた方がよい,というのはあんまりわかっていないです. データのライフサイクルという観点だと,今重要なのは「ビルド前に(操作が)必要か(ビルド時に必要なライブラリのダウンロード)」「ビルド時に必要か(リンクする)」「実行時に必要か(初期化時に読み込む)」という区別が重要だと考えていて,この PR は実行時に必要なモノをすべて |
あー,なるほど.S2E user ごとに管理するべきものと,各シミュレーションケース(ex: |
ということであれば, |
とりあえず Issue 立てました: #613 |
データ関連ファイルは次のような分類になると思います。
これまで、s2e-coreをsubmoduleとして取り込まない場合も想定しており、
そうですね。基本的にはそれに賛成です。 これを気にせずにdataに集約するなら、次のような分類分けが今の構成を自然に拡張できるかなと思っています。
|
@sksat こちら、上の提案でよければ私の方で全て整理しますがいかがでしょうか? |
なるほど,これは特に考えていなかったです & s2e-core を取り込む想定でよいと思います > s2e-coreをsubmoduleとして取り込まない場合も想定 これには明確に反対の立場です.出自とカテゴリが似通っているというだけで,ビルド時に使うモノとランタイムに使うモノはデータのライフサイクルが異なるからです.これらのデータの場所を固定することによって ini ファイル内で s2e-core のディレクトリ構造や実行時のパスからの s2e-core へのパスなどを考慮しないといけなくなってしまうのはユーザ体験が非常に悪いと感じています. >
|
「初期化のためのファイル」みたいな分類をディレクトリ単位でする必要はそんなにないのでは,とは思っています.入力・出力にそれぞれたくさんのファイル・ディレクトリがあるのであればまだしも,出力ファイルは |
逆に ちなみに私としては、シミュレーション設定のためのファイルが一つのディレクトリにまとまっている方が各種説明を行う時に楽ですし、設定のコピペなどもまとまっている方が楽など、ディレクトリ分けした方がメリットがあると思っています。 |
@sksat @suzuki-toshihir0 こちら、だいぶ時間が経ってしまいましたが、どうしましょうか。ここまでで、次のどちらが良いかというのに議論は収束されたかなと思います。
|
AOCSミーティングで出た別の案は、dataディレクトリをなくして階層を浅くしたら良いのではというもの。srcなどと並列して、
|
Related issues
Description
ExtLibraries/GeoPotential
is not library. Just a data.So, it should be move to
data
dir.data/external
dirExtLibraries/GeoPotential
todata/external/geo_potential
scripts/Common/download_EGM96coefficients.sh
intodata/external/geo_potential
EXT_DAYA_DIR_FROM_EXE
virtual environment value for config filesdata/external
files directoryTest results
Provide the test results and a link to the detailed results log.
Impact
Users should be migrate
ExtLibraries/GeoPotential
path reference toEXT_DIR_FROM_EXE/geo_potential