Skip to content

Commit

Permalink
basis default orthonormal, not orthogonal
Browse files Browse the repository at this point in the history
  • Loading branch information
dehann committed Jul 4, 2023
1 parent 2e862bc commit a8e8eaf
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/ParametricManoptDev.jl
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,11 @@ function CalcFactorManopt(fct::DFGFactor, varIntLabel)
if fac_func isa ManifoldPrior
meas = _meas # already a point on M
elseif fac_func isa AbstractPrior
X = get_vector(M, ϵ, _meas, DefaultOrthogonalBasis())
X = get_vector(M, ϵ, _meas, DefaultOrthonormalBasis())
meas = exp(M, ϵ, X) # convert to point on M
else
# its a relative factor so should be a tangent vector
meas = convert(typeof(ϵ), get_vector(M, ϵ, _meas, DefaultOrthogonalBasis()))
meas = convert(typeof(ϵ), get_vector(M, ϵ, _meas, DefaultOrthonormalBasis()))
end

# make sure its an SMatrix
Expand Down Expand Up @@ -97,7 +97,7 @@ struct JacF_RLM!{CF, T}
res::Vector{Float64}
end

function JacF_RLM!(M, costF!; basis_domain::AbstractBasis = DefaultOrthogonalBasis())
function JacF_RLM!(M, costF!; basis_domain::AbstractBasis = DefaultOrthonormalBasis())

p = costF!.points

Expand All @@ -122,7 +122,7 @@ function (jacF!::JacF_RLM!)(
M::AbstractManifold,
J,
p;
basis_domain::AbstractBasis = DefaultOrthogonalBasis(),
basis_domain::AbstractBasis = DefaultOrthonormalBasis(),
)

X0 = jacF!.X0
Expand Down

0 comments on commit a8e8eaf

Please sign in to comment.