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

Fixes for clang on Mac #5

Merged
merged 1 commit into from
May 19, 2019

Conversation

chrisbeall
Copy link
Member

Change FastVector<size_t> to FactorIndices to make clang happy.

This went undetected by Jenkins because https://bitbucket.org/gtborg/gtsam/pull-requests/355 did not have changes from https://bitbucket.org/gtborg/gtsam/pull-requests/409, and gcc did not seem to mind.

@chrisbeall chrisbeall requested a review from dellaert May 19, 2019 02:55
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.

Excellent. Travis I think has a Mac option to prevent this in future...

@chrisbeall chrisbeall merged commit e05e6e9 into borglab:develop May 19, 2019
@chrisbeall chrisbeall deleted the fix/mac-factorIndices branch May 19, 2019 03:07
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.

None yet

2 participants