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

adds stack allocation for registration function() #2

Merged
merged 8 commits into from
Dec 20, 2015

Conversation

giacomodabisias
Copy link

adds stack allocation for registration function()

registration() supports stack memory allocation so a map is added to the class
@giacomodabisias
Copy link
Author

I will work now on the Cmake file to fix the find libfreenect2 issue

@giacomodabisias
Copy link
Author

fixed possibility to open kinect2 by serial

@larshg
Copy link
Owner

larshg commented Dec 20, 2015

Looks great - sorry for the slow response, but I'm busy at work atm. I'll try to add the Cmake required for windows users :)

larshg added a commit that referenced this pull request Dec 20, 2015
adds stack allocation for registration function()
@larshg larshg merged commit ddaa59a into larshg:master Dec 20, 2015
@giacomodabisias
Copy link
Author

Thanks,
let's see if we are missing anything else. I think at the end when everything is fine we need to fix the code for the police style, but that is quick

@giacomodabisias
Copy link
Author

HI,
any news?

larshg pushed a commit that referenced this pull request Jul 9, 2019
larshg pushed a commit that referenced this pull request Feb 4, 2023
…brary#5494)

* PointCloudLibrary#5491

* Fixed typo at line 212 in parse.h

* Fixed formatting.

* Deleted whitespaces.

* Dependency update before attempting fix.

* Fixed formatting style.

* Revert . . formatting?

* Weird formatting error test?

* Fixed dependency, this should hopefully build.

* Removed unused variable in openni_ii_normal_estimation.cpp

* Build test after improvements and refactors.

* Build Test #2.

* deleted whitespace.

* Fixed formatting errors.

* Fixed formatting openni_boundary_estimation.cpp.

* Fixed formatting openni_boundary_estimation.cpp.

* Fixed formatting openni_boundary_estimation.cpp.

* Formatting check #1.

* Formatting check 2.

* Deleted whitespace.

* Changed general parsing structure, improved usage().

* Fixed formatting errors.

* Fixed!

* resolve merge conflict

* resolve merge conflict

* format test

* new format test

* test formatting

* test formatting

* test formatting

* command line behavior mimics previous version(s), consistent help flags

* rest of the files
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