Skip to content
This repository has been archived by the owner on Dec 30, 2023. It is now read-only.

work on normal computing #45

Closed
wants to merge 2 commits into from
Closed

work on normal computing #45

wants to merge 2 commits into from

Conversation

Remi-C
Copy link

@Remi-C Remi-C commented Aug 22, 2014

Fresh pull request from main repository.
Applied the work of colincsl
Corrected it to no more use at function
workaround a Cython bug? can't use regular
T *getptrN(pcl::PointCloud *pc, size_t i)
, even if in htoery with templates it should be okay. Add to duplicate code.

It compiles, I don't know if it works due to local trouble with pcl.

@nzjrs
Copy link
Contributor

nzjrs commented Aug 22, 2014

Please paste the output of 'make test'

@Remi-C
Copy link
Author

Remi-C commented Aug 23, 2014

Hey,
I will do it on monday. For the moment I can't use "import pcl", in a python terminal.
It gives some errors about not finding the _pcl module.
This is quit a handicap ;-)
I must admit I am far beyond my littleknowledge of python and git, let alone templated Cpp and Cython.
Cheers,
Rémi-C

@Remi-C
Copy link
Author

Remi-C commented Aug 25, 2014

Ok,
I can only "import pcl" as root or postgres user.
Even as root or postgres, the "make test" seems faulty.
I have to run "nosetest tests/test.py" to really run the test.

I corrected the test code and the function code to make it work.
Now it passes the tests. (some other functions doesn't pass, but that's not the point here).

It seems I have to create another pull request ...

Cheers,
Rémi-C

@Remi-C Remi-C closed this Aug 25, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants