Skip to content

Hybrid Updates#2056

Merged
varunagrawal merged 4 commits intodevelopfrom
hybrid-updates
Mar 14, 2025
Merged

Hybrid Updates#2056
varunagrawal merged 4 commits intodevelopfrom
hybrid-updates

Conversation

@varunagrawal
Copy link
Copy Markdown
Contributor

Some improvements to the hybrid code to make for easier debugging.

  1. Print the full factors within HybridNonlinearFactor.
  2. Make the involved function functional, rather than use a capture since involvedKeys is later updated.

PS @dellaert can we also get those other PRs merged soon?

@varunagrawal varunagrawal requested a review from dellaert March 12, 2025 21:57
@varunagrawal varunagrawal self-assigned this Mar 12, 2025
@dellaert
Copy link
Copy Markdown
Member

I approved 2 PRs. I prefer if authors with write permission merge their own PRs (and delete the branch).
The exmap PR needs some benchmark numbers in the PR before we can merge. I’ll do that when I get to it.

@varunagrawal varunagrawal merged commit 26a4f70 into develop Mar 14, 2025
36 checks passed
@varunagrawal varunagrawal deleted the hybrid-updates branch March 14, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants