Skip to content

Commit

Permalink
minor cleanup, export number_of_coordinates
Browse files Browse the repository at this point in the history
  • Loading branch information
mateuszbaran committed Nov 8, 2023
1 parent e0b2323 commit 3d14c02
Show file tree
Hide file tree
Showing 5 changed files with 19 additions and 17 deletions.
3 changes: 2 additions & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [0.9.5] - 2023-11-dd
## [0.9.5] - 2023-11-08

### Changed

- `identity_element` now returns a complex matrix for unitary group.
- `number_of_coordinates` is now exported.

## [0.9.4] - 2023-11-06

Expand Down
2 changes: 1 addition & 1 deletion Project.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name = "Manifolds"
uuid = "1cead3c2-87b3-11e9-0ccd-23c62b72b94e"
authors = ["Seth Axen <[email protected]>", "Mateusz Baran <[email protected]>", "Ronny Bergmann <[email protected]>", "Antoine Levitt <[email protected]>"]
version = "0.9.4"
version = "0.9.5"

[deps]
Distributions = "31c24e10-a181-5473-b8eb-7969acd0382f"
Expand Down
1 change: 1 addition & 0 deletions src/Manifolds.jl
Original file line number Diff line number Diff line change
Expand Up @@ -856,6 +856,7 @@ export ×,
norm,
normal_tvector_distribution,
number_eltype,
number_of_coordinates,
one,
power_dimensions,
parallel_transport_along,
Expand Down
2 changes: 1 addition & 1 deletion src/groups/unitary.jl
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ function Unitary(n, 𝔽::AbstractNumbers=ℂ; parameter::Symbol=:type)
end

@doc raw"""
exp_lie(G::Unitary{2,ℂ}, X)
exp_lie(G::Unitary{TypeParameter{Tuple{2}},ℂ}, X)
Compute the group exponential map on the [`Unitary(2)`](@ref) group, which is
Expand Down
28 changes: 14 additions & 14 deletions src/manifolds/GeneralUnitaryMatrices.jl
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ end
check_point(M::GeneralUnitaryMatrices, p; kwargs...)
Check whether `p` is a valid point on the [`UnitaryMatrices`](@ref) or [`OrthogonalMatrices`] `M`,
i.e. that ``p`` has an determinante of absolute value one
i.e. that ``p`` has a determinant of absolute value one
The tolerance for the last test can be set using the `kwargs...`.
"""
Expand Down Expand Up @@ -79,7 +79,7 @@ end
check_point(M::Rotations, p; kwargs...)
Check whether `p` is a valid point on the [`UnitaryMatrices`](@ref) `M`,
i.e. that ``p`` has an determinante of absolute value one, i.e. that ``p^{\mathrm{H}}p``
i.e. that ``p`` has a determinant of absolute value one, i.e. that ``p^{\mathrm{H}}p``
The tolerance for the last test can be set using the `kwargs...`.
"""
Expand Down Expand Up @@ -157,7 +157,7 @@ end
4D rotations can be described by two orthogonal planes that are unchanged by
the action of the rotation (vectors within a plane rotate only within the
plane). The cosines of the two angles $α,β$ of rotation about these planes may be
plane). The cosines of the two angles ``α,β`` of rotation about these planes may be
obtained from the distinct real parts of the eigenvalues of the rotation
matrix. This function computes these more efficiently by solving the system
Expand Down Expand Up @@ -329,18 +329,18 @@ end
get_coordinates(M::OrthogonalMatrices, p, X)
get_coordinates(M::UnitaryMatrices, p, X)
Extract the unique tangent vector components $X^i$ at point `p` on [`Rotations`](@ref)
$\mathrm{SO}(n)$ from the matrix representation `X` of the tangent
Extract the unique tangent vector components ``X^i`` at point `p` on [`Rotations`](@ref)
``\mathrm{SO}(n)`` from the matrix representation `X` of the tangent
vector.
The basis on the Lie algebra $𝔰𝔬(n)$ is chosen such that
for $\mathrm{SO}(2)$, $X^1 = θ = X_{21}$ is the angle of rotation, and
for $\mathrm{SO}(3)$, $(X^1, X^2, X^3) = (X_{32}, X_{13}, X_{21}) = θ u$ is the
angular velocity and axis-angle representation, where $u$ is the unit vector
The basis on the Lie algebra ``𝔰𝔬(n)`` is chosen such that
for ``\mathrm{SO}(2)``, ``X^1 = θ = X_{21}`` is the angle of rotation, and
for ``\mathrm{SO}(3)``, ``(X^1, X^2, X^3) = (X_{32}, X_{13}, X_{21}) = θ u`` is the
angular velocity and axis-angle representation, where ``u`` is the unit vector
along the axis of rotation.
For $\mathrm{SO}(n)$ where $n ≥ 4$, the additional elements of $X^i$ are
$X^{j (j - 3)/2 + k + 1} = X_{jk}$, for $j ∈ [4,n], k ∈ [1,j)$.
For ``\mathrm{SO}(n)`` where ``n ≥ 4``, the additional elements of ``X^i`` are
``X^{j (j - 3)/2 + k + 1} = X_{jk}``, for ``j ∈ [4,n], k ∈ [1,j)``.
"""
get_coordinates(::GeneralUnitaryMatrices{<:Any,ℝ}, ::Any...)
function get_coordinates(
Expand Down Expand Up @@ -476,7 +476,7 @@ end
Convert the unique tangent vector components `Xⁱ` at point `p` on [`Rotations`](@ref)
or [`OrthogonalMatrices`](@ref)
to the matrix representation $X$ of the tangent vector. See
to the matrix representation ``X`` of the tangent vector. See
[`get_coordinates`](@ref get_coordinates(::GeneralUnitaryMatrices, ::Any...)) for the conventions used.
"""
get_vector(::GeneralUnitaryMatrices{<:Any,ℝ}, ::Any...)
Expand Down Expand Up @@ -643,7 +643,7 @@ injectivity_radius(::GeneralUnitaryMatrices) = π
@doc raw"""
injectivity_radius(G::GeneralUnitaryMatrices{<:Any,ℂ,DeterminantOneMatrices})
Return the injectivity radius for general complex unitary matrix manifolds, where the determinant is $+1$,
Return the injectivity radius for general complex unitary matrix manifolds, where the determinant is ``+1``,
which is[^1]
```math
Expand Down Expand Up @@ -726,7 +726,7 @@ Compute the logarithmic map on the [`Rotations`](@ref) manifold
\log_p q = \operatorname{log}(p^{\mathrm{T}}q)
```
where $\operatorname{Log}$ denotes the matrix logarithm. For numerical stability,
where ``\operatorname{Log}`` denotes the matrix logarithm. For numerical stability,
the result is projected onto the set of skew symmetric matrices.
For antipodal rotations the function returns deterministically one of the tangent vectors
Expand Down

0 comments on commit 3d14c02

Please sign in to comment.