code-saturne / code-saturne/code_saturne

Bug: Symmetric tensor diffusivity

オープン
#64 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
307
フォーク
102
PR マージ指標
30日以内にマージされた PR はありません

説明

The combination of `turb_model->iturb = CS_TURB_NONE;` and `iturt(iscalt) = 10` (i.e. GGDH) leads to a segfault. I would expect the code to ignore `iturt` for laminar flow so the user can simply change one parameter when switching on the turbulence model. IMO, we should at least throw a warning that the settings are incompatible, rather than segfault.

It seems the issue stems from `cpltur.f90` where a test for GGDH leads to an assumption of a symmetric tensor diffusivity for the thermal variable, when in reality the diffusivity is a scalar (my settings have diffusivity as variable in space and time, if it makes a difference).

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start in cpltur.f90 and reproduce the segfault with CS_TURB_NONE and iturt(iscalt)=10, including the variable-in-space-and-time diffusivity setting. Trace the GGDH branch for the thermal variable and compare its symmetric-tensor assumption with the laminar scalar case. Done means the incompatible settings no longer segfault and are either safely ignored or reported with a warning.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
fortran
領域
hpc
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

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

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