code-saturne / code-saturne/code_saturne

Bug: Symmetric tensor diffusivity

Aperta
#64 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C++
Stelle
307
Fork
102
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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).

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in cpltur.f90 e riproduci il segfault con CS_TURB_NONE e iturt(iscalt)=10, includendo l'impostazione della diffusività variabile nello spazio e nel tempo. Segui il ramo GGDH per la variabile termica e confronta la sua assunzione di tensore simmetrico con il caso scalare laminare. Il lavoro è completato quando le impostazioni incompatibili non causano più un segfault e vengono ignorate in modo sicuro oppure segnalate con un avviso.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
fortran
Ambito
hpc
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.