Skip to content

Commit

Permalink
Trying to fix FiniteDifferences + SArray failure (#729)
Browse files Browse the repository at this point in the history
* Trying to fix FiniteDifferences + SArray failure

* bump ambiguities :(
  • Loading branch information
mateuszbaran authored Jun 12, 2024
1 parent b0008c0 commit 8602529
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/manifolds/MetricManifold.jl
Original file line number Diff line number Diff line change
Expand Up @@ -487,7 +487,7 @@ function local_metric_jacobian(
backend::AbstractDiffBackend=default_differential_backend(),
)
n = size(p, 1)
∂g = reshape(_jacobian(q -> local_metric(M, q, B), p, backend), n, n, n)
∂g = reshape(_jacobian(q -> local_metric(M, q, B), copy(M, p), backend), n, n, n)
return ∂g
end
@trait_function local_metric_jacobian(
Expand Down
4 changes: 2 additions & 2 deletions test/ambiguities.jl
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ end
# Interims solution until we follow what was proposed in
# https://discourse.julialang.org/t/avoid-ambiguities-with-individual-number-element-identity/62465/2
fmbs = filter(x -> !any(has_type_in_signature.(x, Identity)), mbs)
FMBS_LIMIT = 34
FMBS_LIMIT = 38
println("Number of ManifoldsBase.jl ambiguities: $(length(fmbs))")
@test length(fmbs) <= FMBS_LIMIT
if length(fmbs) > FMBS_LIMIT
Expand All @@ -30,7 +30,7 @@ end
# Interims solution until we follow what was proposed in
# https://discourse.julialang.org/t/avoid-ambiguities-with-individual-number-element-identity/62465/2
fms = filter(x -> !any(has_type_in_signature.(x, Identity)), ms)
FMS_LIMIT = 47
FMS_LIMIT = 53
println("Number of Manifolds.jl ambiguities: $(length(fms))")
if length(fms) > FMS_LIMIT
for amb in fms
Expand Down

0 comments on commit 8602529

Please sign in to comment.