Use extract_unique domain (#291)

* Use extract_unique domain

* Add build requirements install

* Add pytest installation

* Add all ci deps

* Remove another deprecated call

* Yet another deprecated call

Author: jorgensd

.github/workflows/fenicsx-tests.yml

@@ -83,6 +83,7 @@ jobs:
           cmake -G Ninja -DCMAKE_BUILD_TYPE=Developer -B build -S dolfinx/cpp/
           cmake --build build
           cmake --install build
-          python3 -m pip -v install --break-system-packages --no-build-isolation --check-build-dependencies --config-settings=cmake.build-type="Developer" dolfinx/python/
+          python3 -m pip install --break-system-packages -r dolfinx/python/build-requirements.txt
+          python3 -m pip -v install --break-system-packages --no-build-isolation --check-build-dependencies --config-settings=cmake.build-type="Developer" dolfinx/python/[ci]
       - name: Run DOLFINx unit tests
         run: python3 -m pytest -n auto dolfinx/python/test/unit

ufl/pullback.py

@@ -356,7 +356,7 @@ def apply(self, expr):
 
         Returns: The function pulled back to the reference cell
         """
-        domain = expr.ufl_domain()
+        domain = extract_unique_domain(expr)
         space = FunctionSpace(domain, self._element)
         rflat = [expr[idx] for idx in np.ndindex(expr.ufl_shape)]
         g_components = []
@@ -435,7 +435,7 @@ def apply(self, expr):
 
         Returns: The function pulled back to the reference cell
         """
-        domain = expr.ufl_domain()
+        domain = extract_unique_domain(expr)
         space = FunctionSpace(domain, self._element)
         rflat = [expr[idx] for idx in np.ndindex(expr.ufl_shape)]
         g_components = []

ufl/split_functions.py

@@ -7,6 +7,7 @@
 #
 # Modified by Anders Logg, 2008
 
+from ufl.domain import extract_unique_domain
 from ufl.functionspace import FunctionSpace
 from ufl.indexed import Indexed
 from ufl.permutation import compute_indices
@@ -21,7 +22,7 @@ def split(v):
     If v is a Coefficient or Argument in a mixed space, returns a tuple
     with the function components corresponding to the subelements.
     """
-    domain = v.ufl_domain()
+    domain = extract_unique_domain(v)
 
     # Default range is all of v
     begin = 0