precice/calculix-adapter

Inconsistent lookup for read/write data in CHT and FSI

Chiusa

#107 aperta il 24 ott 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)C (30 fork)auto 404
buggood first issue

Metriche repository

Star
 (59 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

For temperature and heat flux, we are checking whether the field name is exactly matched in the config:

https://github.com/precice/calculix-adapter/blob/4496e8f160ec121481653baf44ec032813409ac6/adapter/PreciceInterface.c#L624

For other fields, we are checking whether the field name just starts with this prefix (similar behavior as in the OpenFOAM adapter), sometimes with a trailing -, sometimes without:

https://github.com/precice/calculix-adapter/blob/4496e8f160ec121481653baf44ec032813409ac6/adapter/PreciceInterface.c#L37

https://github.com/precice/calculix-adapter/blob/4496e8f160ec121481653baf44ec032813409ac6/adapter/PreciceInterface.c#L654

Additionally, data names are hard-coded for preCICE:

https://github.com/precice/calculix-adapter/blob/4496e8f160ec121481653baf44ec032813409ac6/adapter/PreciceInterface.c#L628

Finally, this error message is rather confusing:

https://github.com/precice/calculix-adapter/blob/4496e8f160ec121481653baf44ec032813409ac6/adapter/PreciceInterface.c#L662

"Does not exist" made me think that I still needed to specify it somewhere. However, it just means that the adapter does not know how to handle it.

Guida contributor

Direzione di ricerca
Ispeziona il codice in PreciceInterface.c intorno alle righe menzionate. Confronta come vengono abbinati i nomi dei campi per diversi tipi di dati (temperatura/flusso di calore vs altri). Identifica l'incoerenza e i nomi dei dati hard coded. Traccia il messaggio di errore per capire quando viene attivato. Determina il comportamento previsto per i casi CHT (trasferimento di calore coniugato) e FSI (interazione fluido struttura). Proponi un metodo uniforme per l'abbinamento dei nomi dei campi e migliora il messaggio di errore per renderlo più chiaro.
Tech stack
c
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitC
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70