Skip to content

Fix the build for example Hybrid City 10000#2064

Merged
varunagrawal merged 1 commit intoborglab:developfrom
jmackay2:build_fix
Mar 22, 2025
Merged

Fix the build for example Hybrid City 10000#2064
varunagrawal merged 1 commit intoborglab:developfrom
jmackay2:build_fix

Conversation

@jmackay2
Copy link
Copy Markdown
Contributor

Issue:

When building, I get the following error:

gtsam/examples/Hybrid_City10000.cpp:113:35: error: cannot convert ‘size_t’ {aka ‘long unsigned int’} to ‘const gtsam::Values&’
  113 |     smoother_.update(*linearized, maxNrHypotheses);
      |                                   ^~~~~~~~~~~~~~~
      |                                   |
      |                                   size_t {aka long unsigned int}

Instead of using the size_t, this changes it to use the Values called initial_.

@dellaert dellaert requested a review from varunagrawal March 21, 2025 00:55
Copy link
Copy Markdown
Contributor

@varunagrawal varunagrawal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This actually needs some more work but that should be my responsibility. Thanks!

@varunagrawal varunagrawal merged commit b8dbcba into borglab:develop Mar 22, 2025
36 checks passed
@jmackay2 jmackay2 deleted the build_fix branch April 9, 2025 02:47
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