Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Miscellaneous pre-existing issues found while reviewing #45

オープン
#47 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

@manishvenu がすでに取り組んでいます。

2026年9月11日 から。

評価

この issue はまだ評価されていません。

説明

Three unrelated issues that came up while reviewing #45. None of them are introduced by that PR, so noting them here instead.

  1. ww3_input_generator.py:80 disables the Generate button for the duration of on_btn_generate_clicked, and restores it at lines 92 and 102, i.e., on the two error paths only. The success path at line 99 doesn't restore it, and reset() (lines 49-52) clears WW3_INPUT_STATUS without re-enabling the button. So after a successful generation, returning to the Wave Grid stage leaves it blocked on a None status, with the only button that can set that status disabled. Adding self.disabled = False after line 99, or moving the restore into reset(), should cover it.

  2. case_creator.py:44:

assert is_ccs_config_writeable or not add_grids_to_ccs_config, "Cannot write to ccs_config xml files. ..."

is_ccs_config_writeable is a function (case_tools.py:12), so the bare name is always truthy and this assert can never fail. It is called correctly at line 276. Should be is_ccs_config_writeable(cime).

  1. case_creator.py:717 and 722, in _apply_mom_namelist_changes, open two datasets that are never closed, and the vgrid one isn't even bound to a name:
ds_topo = xr.open_dataset(topo_file_path)
...
nk = len(xr.open_dataset(vgrid_file_path).dz)

Both should be with xr.open_dataset(...) as ds:.

主要言語
Python
スター
16
フォーク
4
平均マージ
9時間
マージ済み PR(30日)
2

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ESMCI/visualCaseGen のほかの issue

ESMCI/visualCaseGen の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。