Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

channel-transport-reaction triggers assertion in fenics-adapter #374

Open
BenjaminRodenberg opened this issue Sep 28, 2023 · 2 comments
Open
Assignees

Comments

@BenjaminRodenberg
Copy link
Member

The following assertion is triggeres by channel-transport-reaction:

tutorials/channel-transport-reaction/fluid-fenics$ ./run.sh 
Calling FFC just-in-time (JIT) compiler, this may take some time.
Calling FFC just-in-time (JIT) compiler, this may take some time.
Calling FFC just-in-time (JIT) compiler, this may take some time.
---[precice]  This is preCICE version 3.0.0
---[precice]  Revision info: v2.5.0-461-ge1e20f9d6
---[precice]  Build type: Debug
---[precice]  Configuring preCICE with configuration "/home/benjamin/Programming/tutorials/channel-transport-reaction/fluid-fenics/../precice-config.xml"
---[precice]  I am participant "Flow"
Participant Flow is write-only participant
Traceback (most recent call last):
  File "fluid.py", line 121, in <module>
    precice.initialize(coupling_subdomain=CouplingDomain(), write_object=u_)
  File "/home/benjamin/.local/lib/python3.8/site-packages/fenicsprecice/fenicsprecice.py", line 419, in initialize
    vertices = get_coupling_triangles(function_space, coupling_subdomain, fenics_edge_ids, id_mapping)
  File "/home/benjamin/.local/lib/python3.8/site-packages/fenicsprecice/adapter_core.py", line 439, in get_coupling_triangles
    assert (v2 != v2)
AssertionError
---[precice]  Implicitly finalizing in destructor

Since I'm not familiar with this tutorial, I will document this issue for now in favor of merging #333. Any help here would be appreciated. First starting point would be to check whether this tutorial actually works on the latest distribution release.

@BenjaminRodenberg
Copy link
Member Author

The assertion is obviously wrong, but the function is also not tested in the adapter. This makes it basically an issue of the adapter that we also have to fix over there. Cannot evaluate the tutorial, but for now I will assume that it is working. Let's keep this issue open until the problem in the adapter is fixed.

@BenjaminRodenberg
Copy link
Member Author

The partitioned heat equation actually runs into the same problem. This was just hidden by the way how the run.sh was set up. Changed this, see 5d22b76.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants