Skip to content

Commit

Permalink
GLT symbol of bilinear forms containing boundary integrals (#122)
Browse files Browse the repository at this point in the history
The GLT symbol was not computed correctly when a bilinear form contained a boundary integral. The desired behaviour is:

- `gelatize` of a bilinear form involving only boundary integrals should return zero.
- in the case of having a bilinear form involving the sum of boundary integrals and domain integrals, `gelatize` should only be applied to the sum of domain integrals.
  • Loading branch information
e-moral-sanchez authored Jun 28, 2023
1 parent c3a5993 commit fdc7c4d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions sympde/expr/evaluation.py
Original file line number Diff line number Diff line change
Expand Up @@ -1140,7 +1140,7 @@ def eval(cls, *_args, **kwargs):

return ImmutableDenseMatrix(lines)

elif isinstance(expr, DomainExpression):
elif isinstance(expr, (DomainExpression, BoundaryExpression)):
# TODO to be removed
return cls.eval(expr.expr, d_atoms=d_atoms, domain=domain)

Expand All @@ -1160,7 +1160,7 @@ def eval(cls, *_args, **kwargs):
# ...

# ...
if domain is not None and domain.mapping is not None:
if domain is not None and getattr(domain, 'mapping', None):
terminal_expr = LogicalExpr(terminal_expr.expr, domain)
variables = [LogicalExpr(e, domain) for e in variables ]
trials = [LogicalExpr(e, domain) for e in trials ]
Expand Down
2 changes: 1 addition & 1 deletion sympde/version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.17.0"
__version__ = "0.17.1"

0 comments on commit fdc7c4d

Please sign in to comment.