precice/openfoam-adapter

Remove the connectivity switch from the configuration, ask preCICE

Aperta

#198 aperta il 20 set 2021

Vedi su GitHubGuida contributor
 (1 commento) (1 reazione) (0 assegnatari)C++ (115 fork)auto 404
compatibilityenhancementgood first issue

Metriche repository

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

Descrizione

preCICE v2.3.0 https://github.com/precice/precice/pull/1080 introduces an API method to ask whether mesh connectivity is required. In the OpenFOAM adapter, we currently ask the user whether this is required, which the user should not need to configure, as it is system-internal information: https://precice.org/adapter-openfoam-config.html#nearest-projection-mapping

We could directly ask preCICE for this and remove the option from the configuration. As it currently defaults to false, maybe we can do it in a backwards-compatible way and deprecate the option for now.

Guida contributor

Direzione di ricerca
Controlla il metodo API di preCICE per richiedere il requisito di connettività della mesh, quindi rimuovi l'opzione di configurazione e aggiungi un avviso di deprecazione.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75