-
Notifications
You must be signed in to change notification settings - Fork 191
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
[jsk_pcl/OrganizedStatisticalOutlierRemoval] support several pointcloud type #2846
base: master
Are you sure you want to change the base?
Conversation
? What happened to CI?
|
a3dcddd
to
38446c6
Compare
https://github.com/jsk-ros-pkg/jsk_recognition/actions/runs/11564823139/job/32190755001?pr=2846 |
😢 |
38446c6
to
799ff8e
Compare
|
f088f31
to
ca47dd2
Compare
Some people point out the same issue. I also can't find the official announcement... https://github.com/orgs/community/discussions/145711 |
https://github.blog/changelog/2024-09-25-end-of-life-for-actions-node16/ I think we cannot use |
😭 |
as my understanding, we use jenkins for jsk_recognition CI processes, so that GA container's version does not matter for CI processes. |
33ccf4c
to
887ad0a
Compare
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.
@knorth55
could you resolve conflicts?
887ad0a
to
da7f974
Compare
@k-okada Hi, I resolved the conflict by rebasing this PR. |
this PR supports several point cloud type as
xyzirt
.now this nodes only support
xyzrgb
.