Skip to content
New issue

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

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

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants