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

Supervoxel coloring update #941

Merged

Conversation

jpapon
Copy link
Contributor

@jpapon jpapon commented Oct 2, 2014

This changes supervoxel clustering so that it uses the changes from #849 (new label color handler).
The functions for getting RGB clouds have been deprecated, and the example updated.
The label color handler was also updated so that it uses modulo the max label, rather than random colors. This fixes the problem where colors change if the cloud is updated and is faster.

Also, spacing in the header file was fixed so git doesn't complain anymore.

@taketwo
Copy link
Member

taketwo commented Oct 2, 2014

I would keep getRandomColor() function, might be useful in other situations.

@jpapon jpapon force-pushed the supervoxel_coloring_update branch from 4e2238d to 62f0d7e Compare October 2, 2014 12:22
@jpapon
Copy link
Contributor Author

jpapon commented Oct 2, 2014

Done.

pcl::io::PointCloudImageExtractorFromLabelField<PointLT> pcie (pcie.io::PointCloudImageExtractorFromLabelField< PointLT>::COLORS_RGB_GLASBEY);
pcl::PCLImage image;
pcie.extract (*full_labeled_cloud, image);
pcl::io::savePNGFile(out_path, image);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need space here and below.

@taketwo
Copy link
Member

taketwo commented Oct 3, 2014

Can you squash the fixup commits? Then I'll merge.

…use the label field handlers from PointCloudLibrary#849 , deprecated functions return empty cloud to deprecated functions to avoid warnings

Changed labeled color handler to use modulo max color instead of using random colors
@jpapon jpapon force-pushed the supervoxel_coloring_update branch from 46cd0e5 to 81e8664 Compare October 3, 2014 09:04
@jpapon
Copy link
Contributor Author

jpapon commented Oct 3, 2014

gesquashed

taketwo added a commit that referenced this pull request Oct 6, 2014
@taketwo taketwo merged commit eed64d0 into PointCloudLibrary:master Oct 6, 2014
@SergioRAgostinho SergioRAgostinho mentioned this pull request Dec 12, 2015
7 tasks
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