-
Notifications
You must be signed in to change notification settings - Fork 22
Closed
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers
Description
Describe the issue:
The enrollment of the old (deprecated) FluxBoundary function is not working. More precisely, even though I do the enrollment in setup.cpp the function never gets called up in the main loop. This is because of this line
idefix/src/fluid/boundary/boundary.hpp
Line 202 in 2f15373
| this->haveUserDefBoundary = true; |
Here is the fix:
diff --git a/src/fluid/boundary/boundary.hpp b/src/fluid/boundary/boundary.hpp
index 126126a..273bcf6 100644
--- a/src/fluid/boundary/boundary.hpp
+++ b/src/fluid/boundary/boundary.hpp
@@ -200,7 +200,7 @@ void Boundary<Phys>::EnrollFluxBoundary(UserDefBoundaryFuncOld myFunc) {
IDEFIX_WARNING(msg);
this->fluxBoundaryFuncOld = myFunc;
- this->haveUserDefBoundary = true;
+ this->haveFluxBoundary = true;
}
Error message:
No error messageruntime information:
This is on the master branch of the commit 2f15373.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers