Number KMV by subdegree, not superdegree (#274)

* renumber kmv by subdegree * update test list
mscroggs · Sep 25, 2024 · 567497e · 567497e
1 parent 52637f3
commit 567497e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/symfem/elements/kmv.py b/symfem/elements/kmv.py
@@ -88,7 +88,7 @@ def __init__(self, reference: Reference, order: int):
                     )
                 poly = kmv_tri_polyset(1, 1)
 
-            elif order == 3:
+            elif order == 2:
                 for v_n, v in enumerate(reference.vertices):
                     dofs.append(
                         WeightedPointEvaluation(
@@ -115,7 +115,7 @@ def __init__(self, reference: Reference, order: int):
                 )
                 poly = kmv_tri_polyset(2, 3)
 
-            elif order == 4:
+            elif order == 3:
                 for v_n, v in enumerate(reference.vertices):
                     dofs.append(
                         WeightedPointEvaluation(
@@ -177,7 +177,7 @@ def __init__(self, reference: Reference, order: int):
                     )
                 poly = kmv_tet_polyset(1, 1, 1)
 
-            elif order == 4:
+            elif order == 2:
                 for v_n, v in enumerate(reference.vertices):
                     dofs.append(
                         WeightedPointEvaluation(

diff --git a/test/utils.py b/test/utils.py
@@ -54,7 +54,7 @@
         "BDFM": [({"variant": "equispaced"}, range(1, 4))],
         "Argyris": [({}, [5])],
         "MTW": [({"variant": "equispaced"}, [3])],
-        "KMV": [({}, [1, 3])],
+        "KMV": [({}, [1, 2])],
         "Hermite": [({}, [3])],
         "BDM": [({"variant": "equispaced"}, range(1, 4))],
         "Bernstein": [({}, range(1, 4))],