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.

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

Mac compiler fixes and GeographicLib upgrade #6

Merged
merged 3 commits into from
May 19, 2019

Conversation

chrisbeall
Copy link
Member

  • Fix more instances of FactorIndices issues in tests on Mac.
  • GeographicLib configuration improvements. Toggling GTSAM_INSTALL_GEOGRAPHICLIB from ON to OFF resulted in build errors. Now it works.
  • GeographicLib upgrade. Recently started getting warnings about i386 build being deprecated on Mac, and linker errors. Upgrade fixes these issues, and all tests pass on Mac.

@chrisbeall chrisbeall requested a review from dellaert May 19, 2019 04:16
Copy link
Member

@dellaert dellaert left a comment

Choose a reason for hiding this comment

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

Will trust you on this :-)

@ghost
Copy link

ghost commented May 19, 2019

Verified with GTSAM_INSTALL_GEOGRAPHICLIB=ON on Ubuntu 16.04. All 236 tests passed. I'll merge.

@chrisbeall chrisbeall merged commit 690b2b8 into borglab:develop May 19, 2019
@chrisbeall chrisbeall deleted the fix/compiler-and-cmake-mac branch May 19, 2019 04:31
varunagrawal added a commit that referenced this pull request Sep 16, 2020
314b121fd Merge pull request #6 from varunagrawal/feature/pybind-upgrade
5e49bb867 Merge commit '62e790da1cc53cb9910ac5271a3514d88562bdce' into feature/docstring
62e790da1 Squashed 'pybind11/' changes from 441e777..d3c999c
e51526702 Merge pull request #5 from varunagrawal/feature/docstring
f503ddb2d document use_boost
2bb7158fc complete arguments documentation for pybind_wrap function

git-subtree-dir: wrap
git-subtree-split: 314b121fd4017338a3a6833728cd646d8ff5be12
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