Skip to content

Commit

Permalink
Specialise only on DefaultOrthogonalBasis
Browse files Browse the repository at this point in the history
  • Loading branch information
Affie committed Oct 16, 2023
1 parent c504d10 commit 1bbb150
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/groups/semidirect_product_group.jl
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ end
# We need to prevent decorator unwrapping so that the correct `get_vector!` gets called
# and applies proper padding to the result if `X` happens to be a matrix.
# Otherwise rare random bugs happen where the padding is not applied.
function get_vector(G::SemidirectProductGroup, p, X, B::AbstractOrthogonalBasis)
function get_vector(G::SemidirectProductGroup, p, X, B::VeeOrthogonalBasis)
Y = allocate_result(G, get_vector, p, X)
return get_vector!(G, Y, p, X, B)
end
Expand Down
6 changes: 3 additions & 3 deletions src/groups/special_euclidean.jl
Original file line number Diff line number Diff line change
Expand Up @@ -717,7 +717,7 @@ function get_coordinates(
M::SpecialEuclidean,
p::ArrayPartition,
X::ArrayPartition,
basis::AbstractBasis,
basis::DefaultOrthogonalBasis,
)
M1, M2 = M.manifold.manifolds
return vcat(

Check warning on line 723 in src/groups/special_euclidean.jl

View check run for this annotation

Codecov / codecov/patch

src/groups/special_euclidean.jl#L722-L723

Added lines #L722 - L723 were not covered by tests
Expand All @@ -736,7 +736,7 @@ function get_vector(
M::SpecialEuclidean{2},
p::ArrayPartition,
c::AbstractVector,
basis::AbstractOrthogonalBasis,
basis::DefaultOrthogonalBasis,
)
return ArrayPartition(

Check warning on line 741 in src/groups/special_euclidean.jl

View check run for this annotation

Codecov / codecov/patch

src/groups/special_euclidean.jl#L741

Added line #L741 was not covered by tests
get_vector(M.manifold.manifolds[1].manifold, p.x[1], view(c, 1:2), basis),
Expand All @@ -754,7 +754,7 @@ function get_vector(
M::SpecialEuclidean{3},
p::ArrayPartition,
Xc::AbstractVector,
basis::AbstractOrthogonalBasis,
basis::DefaultOrthogonalBasis,
)
return ArrayPartition(

Check warning on line 759 in src/groups/special_euclidean.jl

View check run for this annotation

Codecov / codecov/patch

src/groups/special_euclidean.jl#L759

Added line #L759 was not covered by tests
get_vector(M.manifold.manifolds[1].manifold, p.x[1], view(Xc, 1:3), basis),
Expand Down

0 comments on commit 1bbb150

Please sign in to comment.