Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

_solveLambdaNumeric for AbstractManifoldMinimize does not work for deconv #1374

Closed
Affie opened this issue Aug 30, 2021 · 2 comments · Fixed by #1380
Closed

_solveLambdaNumeric for AbstractManifoldMinimize does not work for deconv #1374

Affie opened this issue Aug 30, 2021 · 2 comments · Fixed by #1380
Assignees
@Affie
Labels
bug decision design manifolds sampling
Milestone
v0.25.0

Comments

@Affie
Copy link
Member

Affie commented Aug 30, 2021

No description provided.

@Affie Affie added this to the v0.25.0 milestone Aug 30, 2021
@dehann
Copy link
Member

dehann commented Sep 9, 2021 •
edited
Loading

Short term, implement a secondary _solveLambdaNumericDeconv specific to <:AbstractManifoldMinimize.

Problem is current _solveLambdaNumeric assumes it will only solve for variable points, does not include need for solving measurement tangents.

All this will eventually need to be consolidated back to the new <:AbstractManifoldMinimize as was previously consolidated with other <:AbstractRelatives.

@Affie
Copy link
Member Author

Affie commented Sep 9, 2021 •
edited by dehann
Loading

For IIF v0.25.0 only measurements as tangents can be solved with deconvolution.

xref JuliaRobotics/RoME.jl#465

@Affie Affie self-assigned this Sep 9, 2021
@Affie Affie mentioned this issue Sep 9, 2021
Merged
@Affie Affie mentioned this issue Sep 10, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Affie Affie

Labels
bug decision design manifolds sampling
Projects
None yet
Milestone
v0.25.0
Development

Successfully merging a pull request may close this issue.

Fix and test manifold approxDeconv JuliaRobotics/IncrementalInference.jl
2 participants
@dehann @Affie