Merge #1366

1366: Change Ref -> tuple in broadcasting CartesianVector r=charleskawczynski a=charleskawczynski This PR changes Ref -> tuple in broadcasting CartesianVector, since we were having allocation issues with `Ref`. Co-authored-by: Charles Kawczynski <[email protected]>
CliMA · Jul 13, 2023 · 445448e · 445448e
2 parents 68fabae + 3edb5fd
commit 445448e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/Fields/broadcast.jl b/src/Fields/broadcast.jl
@@ -419,7 +419,7 @@ function Base.Broadcast.broadcasted(
         fs,
         V,
         arg,
-        Ref(space.global_geometry),
+        tuple(space.global_geometry),
         local_geometry_field(space),
     )
 end

diff --git a/src/Operators/spectralelement.jl b/src/Operators/spectralelement.jl
@@ -312,6 +312,14 @@ Base.@propagate_inbounds _get_node(space, ij, slabidx, arg, xargs...) = (
 Base.@propagate_inbounds function get_node(space, scalar, ij, slabidx)
     scalar[]
 end
+Base.@propagate_inbounds function get_node(
+    space,
+    scalar::Tuple{<:Any},
+    ij,
+    slabidx,
+)
+    scalar[1]
+end
 Base.@propagate_inbounds function get_node(
     parent_space,
     field::Fields.Field,