-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added RTree3D wrapper #21
base: master
Are you sure you want to change the base?
Conversation
gavincangan
commented
Apr 17, 2019
- Added a RTree3D class similar to the existing RTree
- Added simple tests for RTree3D functions in a jupyter notebook in ./tests (I have more tests in my other project that I am using them in)
Former-commit-id: 07c4a789dceee0cd71affc5371abf867f6c78b34
Former-commit-id: 8cb3dd68bffbfe77e6fb97f2e5747504612e0567
Former-commit-id: 0707a4783156b50cb574caee1ad22e1702d7df18
Please remove all |
|
setup.py
Outdated
@@ -39,6 +39,7 @@ | |||
print('| Ubuntu: apt-get install libboost-all-dev |') | |||
print('| Manually: wget https://sourceforge.net/projects/boost/files/boost/1.65.1/boost_1_65_1.tar.gz|') | |||
print('=' * 40) | |||
os.environ['BOOST_ROOT'] = '/usr/include' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :)
setup.py
Outdated
@@ -62,7 +63,7 @@ | |||
sources=sources, | |||
language="c++", | |||
extra_compile_args=compile_args, | |||
include_dirs=[numpy.get_include(), os.environ['BOOST_ROOT'], 'include'] | |||
include_dirs=[numpy.get_include(), os.environ['BOOST_ROOT'], 'include', '/usr/include', '/usr/include/x86_64-linux-gnu'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can set BOOST_ROOT=/usr/include/x86_64-linux-gnu
here, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed 👍
src/RTreePoint3D.cpp
Outdated
|
||
void RTreePoint3D::insertPoints(double* points, long m, long n) | ||
{ | ||
assert(n == 4); // points should be an m x 3 matrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update comment, it is m x 4
now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed 👍
…th manually, which obviously does not work well