Skip to content

Commit f3fed3d

Browse files
authored
Fix #685 (#686)
* Add test that should segfault on oneAPI * flake * Use blas
1 parent 42402da commit f3fed3d

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

cpp/basix/math.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -310,9 +310,9 @@ template <typename U, typename V, typename W>
310310
void dot(const U& A, const V& B, W&& C)
311311
{
312312
assert(A.extent(1) == B.extent(0));
313-
assert(C.extent(0) == C.extent(0));
313+
assert(C.extent(0) == A.extent(0));
314314
assert(C.extent(1) == B.extent(1));
315-
if (A.extent(0) * B.extent(1) * A.extent(1) < 4096)
315+
if (A.extent(0) * B.extent(1) * A.extent(1) < 512)
316316
{
317317
std::fill_n(C.data_handle(), C.extent(0) * C.extent(1), 0);
318318
for (std::size_t i = 0; i < A.extent(0); ++i)

test/test_create.py

+5
Original file line numberDiff line numberDiff line change
@@ -82,3 +82,8 @@ def test_create_element(cell, degree, family, variant):
8282
except RuntimeError as e:
8383
if len(e.args) == 0 or "dgesv" in e.args[0]:
8484
raise e
85+
86+
87+
def test_create_high_degree_lagrange():
88+
basix.create_element(
89+
basix.ElementFamily.P, basix.CellType.hexahedron, 7, basix.LagrangeVariant.gll_isaac)

0 commit comments

Comments
 (0)