Skip to content

Add check for mesh connectivity requirement before computing mesh edges#138

Closed
IshaanDesai wants to merge 1 commit intodevelopfrom
add-MeshConnectivity-bool
Closed

Add check for mesh connectivity requirement before computing mesh edges#138
IshaanDesai wants to merge 1 commit intodevelopfrom
add-MeshConnectivity-bool

Conversation

@IshaanDesai
Copy link
Copy Markdown
Member

In precice/precice#1080 the API function isMeshConnectivityRequired() was added to check if mesh connectivity information is required by preCICE. This API function is now used to check whether the mesh edge computation is necessary.

@IshaanDesai IshaanDesai self-assigned this Sep 16, 2021
@IshaanDesai IshaanDesai added the enhancement New feature or request label Sep 16, 2021
@uekerman
Copy link
Copy Markdown
Member

Please be aware that this means that the FEniCS adapter can no longer be used with preCICE < v2.3.

@IshaanDesai IshaanDesai removed the request for review from BenjaminRodenberg September 20, 2021 12:20
@IshaanDesai IshaanDesai marked this pull request as draft September 20, 2021 12:21
@BenjaminRodenberg
Copy link
Copy Markdown
Contributor

Please be aware that this means that the FEniCS adapter can no longer be used with preCICE < v2.3.

Is it possible to only perform this check, if the preCICE version is recent enough and skip the check (and raise a warning?) if preCICE is < 2.3? pyprecice allows you to query the version of the bindings. So you should be able to check the version and act correspondingly inside the adapter.

@IshaanDesai
Copy link
Copy Markdown
Member Author

Check added in #153

@IshaanDesai IshaanDesai deleted the add-MeshConnectivity-bool branch March 28, 2023 19:20
BenjaminRodenberg added a commit that referenced this pull request Sep 28, 2023
* Remove API functions that are removed in v3.0.0. Fix data initialization.
* Forward dt to python bindings.
* Set mesh connectivity information only if it is required. Closes #138
* Update tests
* Update links of contributors
* Update CHANGELOG.md
* Call checkpointing functions from the interface
* Change required version of pyprecice to >=3.0.0.0
* Comment out nearest projection volume coupling.

---------

Co-authored-by: Ishaan Desai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants