precice/calculix-adapter

Inconsistent lookup for read/write data in CHT and FSI

Aberta

#107 aberto em 24 de out. de 2022

Ver no GitHub
 (0 comentário) (0 reação) (1 responsável)C (30 forks)auto 404
buggood first issue

Métricas do repositório

Stars
 (59 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.

Guia do colaborador

Direção de pesquisa
Inspecione o código em PreciceInterface.c ao redor das linhas mencionadas. Compare como os nomes de campos são correspondidos para diferentes tipos de dados (temperatura/fluxo de calor vs outros). Identifique a inconsistência e os nomes de dados codificados. Rastreie a mensagem de erro para entender quando ela é acionada. Determine o comportamento pretendido para os casos CHT (transferência de calor conjugada) e FSI (interação fluido estrutura). Proponha um método uniforme para correspondência de nomes de campos e melhore a mensagem de erro para ficar mais clara.
Pilha de tecnologia
c
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitC
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70