Only build MARBL if requested in compset longname / alias
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- xml
- 領域
- build-system, testing
調査の方向性
env_build.xml 内の MOM6_BUILD_MARBL エントリを特定し、config_component.xml の関連する変更を確認します。no_marbl test-mod ディレクトリと、それを使用するテストを確認します。非 MARBL compset がデフォルトでスタブを使用し、MARBL compset がライブラリと関連設定を有効にし、古い no_marbl テスト設定が削除されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Currently MARBL is built by default, regardless of whether the user requested it. We should switch to only building it when the compset specifies running MOM6 with the MARBL tracers:
<entry id="MOM6_BUILD_MARBL">
<type>char</type>
<valid_values>TRUE,FALSE</valid_values>
- <default_value>TRUE</default_value>
+ <default_value>FALSE</default_value>
+ <values>
+ <value compset="_MOM6%[^_]*MARBL">TRUE</value>
+ </values>
<group>build_component_mom</group>
<file>env_build.xml</file>
<desc> This variable controls whether MOM6 builds with the full MARBL library (default) or just
with stubs. If set to FALSE then setting USE_MARBL_TRACERS = True in the Parameters file
- will result in a runtime error. It is recommended to leave this variable set to TRUE, it
- exists to allow testing of the MARBL stub in the CESM test suite.
+ will result in a runtime error.
</desc>
</entry>
We don't want users to start with a G compset and try to add MARBL after the fact. The default PE layouts for compsets without MARBL are much smaller than the comparable layouts with MARBL enabled, so adding MARBL after create_newcase will run much slower unless the user changes NTASKS_OCN before building. Also there are a handful of env_run.xml changes that come through config_component.xml that would need to be set manually:
OCN_CO2_TYPEMARBL_TRACER_OPTSMARBL_CONFIGMARBL_DIAG_MODE
A side benefit is that this would allow us to remove cime_config/testdefs/testmods_dirs/mom/no_marbl/ and the lone test using that directory in the test suite because every non-MARBL compset would test the MARBL stub rather than the full library.
- 主要言語
- Jupyter Notebook
- スター
- 6
- フォーク
- 23
- 平均マージ
- 2日 22時間
- マージ済み PR（30日）
- 4
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ESCOMP/MOM_interface のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
ESCOMP/MOM_interface#352 ·
-
Default value for ATM_ALT_CO2_OPT should be const対応中かも @mnlevy1981 が 4 日前に担当しました。 オープン
ESCOMP/MOM_interface#351 · コメント 1 件 · 担当者 1 名 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
ESCOMP/MOM_interface#323 ·
-
Error for missing INPUT files再び着手できるかも @alperaltuntas が 182 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
ESCOMP/MOM_interface#317 · 担当者 1 名 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
ESCOMP/MOM_interface#310 ·
ESCOMP/MOM_interface の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
trezor/trezor-firmware#7997 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
bug good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
score-spec/docs#329 · コメント 1 件 ·
-
package-update
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
oSoWoSo/vOid_Community_repOsitory#207 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
vllm-project/vllm-ascend#17784 ·
メンテナーはふだん 1 日以内に返信