Reintroduce pointcloud render without Float_V message by arjo129 · Pull Request #755 · gazebosim/gz-gui

arjo129 · 2026-03-17T03:29:09Z

🦟 Bug fix

Summary

Re-introduce point cloud rendering without the Float_V message.

I tested it with the following program:

#include <iostream>
#include <cmath>
#include <thread>
#include <chrono>
#include <gz/msgs/pointcloud_packed.pb.h>
#include <gz/msgs/PointCloudPackedUtils.hh>
#include <gz/transport/Node.hh>

int main(int argc, char** argv)
{
  // Create a transport node.
  gz::transport::Node node;

  // Define the topic to publish on.
  std::string topic = "/sphere_points";

  // Create a publisher for PointCloudPacked messages.
  auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic);

  if (!pub)
  {
    std::cerr << "Error advertising topic [" << topic << "]" << std::endl;
    return -1;
  }

  // Sphere parameters
  double radius = 1.0;
  int num_theta = 20;
  int num_phi = 40;

  // Prepare the message
  gz::msgs::PointCloudPacked msg;
  // Initialize header
  msg.mutable_header()->mutable_stamp()->set_sec(0);
  msg.mutable_header()->mutable_stamp()->set_nsec(0);

  // Initialize PointCloudPacked
  gz::msgs::InitPointCloudPacked(msg, "frame_id", false,
      {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}});

  int num_points = (num_theta + 1) * num_phi;
  msg.set_width(num_points);
  msg.set_height(1);
  msg.set_is_dense(true);
  msg.mutable_data()->resize(num_points * msg.point_step());

  gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x");
  gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y");
  gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z");

  const double PI = std::acos(-1.0);

  for (int i = 0; i <= num_theta; ++i)
  {
    double theta = PI * i / num_theta;
    for (int j = 0; j < num_phi; ++j)
    {
      double phi = 2.0 * PI * j / num_phi;

      double x = radius * std::sin(theta) * std::cos(phi);
      double y = radius * std::sin(theta) * std::sin(phi);
      double z = radius * std::cos(theta);

      *iterX = static_cast<float>(x);
      *iterY = static_cast<float>(y);
      *iterZ = static_cast<float>(z);
      ++iterX;
      ++iterY;
      ++iterZ;
    }
  }

  std::cout << "Publishing a sphere with " << num_points
            << " points on topic [" << topic << "]" << std::endl;

  // Publish the message in a loop
  while (true)
  {
    if (!pub.Publish(msg))
    {
        std::cerr << "Failed to publish message" << std::endl;
    }
    std::this_thread::sleep_for(std::chrono::milliseconds(1000));
  }

  return 0;
}

TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in gz-math. For now there is rudimentary support for colors in this PR.

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

…pointcloud_render

…issing Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

Co-authored-by: Ian Chen <ichen@openrobotics.org> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com>

Co-authored-by: Ian Chen <iche@google.com> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com>

arjo129 · 2026-03-18T20:17:36Z

@Mergifyio backport gz-sim10 gz-sim9 gz-sim8 ign-gazebo6

mergify · 2026-03-18T20:17:47Z

backport gz-sim10 gz-sim9 gz-sim8 ign-gazebo6

❌ No backport have been created

Details

Backport to branch gz-sim10 failed

GitHub error: Branch not found

Backport to branch gz-sim9 failed

GitHub error: Branch not found

Backport to branch gz-sim8 failed

GitHub error: Branch not found

Backport to branch ign-gazebo6 failed

GitHub error: Branch not found

azeey · 2026-03-18T21:49:06Z

https://github.com/Mergifyio backport gz-sim10 gz-sim9 gz-sim8 ign-gazebo6

mergify · 2026-03-18T21:49:32Z

backport gz-sim10 gz-sim9 gz-sim8 ign-gazebo6

❌ No backport have been created

Details

Backport to branch gz-sim10 failed

GitHub error: Branch not found

Backport to branch gz-sim9 failed

GitHub error: Branch not found

Backport to branch gz-sim8 failed

GitHub error: Branch not found

Backport to branch ign-gazebo6 failed

GitHub error: Branch not found

azeey · 2026-03-18T21:56:14Z

@Mergifyio backport gz-gui10 gz-gui8 gz-gui9 ign-gui6

@arjo129 oops, I did the same thing you did thinking it was a github outage. Btw, check out https://gazebosim.org/docs/latest/maintainers/#backporting for quickly generating the mergifyio command for a given PR.

mergify · 2026-03-18T21:56:41Z

backport gz-gui10 gz-gui8 gz-gui9 ign-gui6

✅ Backports have been created

Details

#756 Reintroduce pointcloud render without Float_V message (backport #755) has been created for branch gz-gui10
#757 Reintroduce pointcloud render without Float_V message (backport #755) has been created for branch gz-gui8 but encountered conflicts

Cherry-pick of d8b6a54 has failed:

On branch mergify/bp/gz-gui8/pr-755
Your branch is up to date with 'origin/gz-gui8'.

You are currently cherry-picking commit d8b6a54.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   src/plugins/point_cloud/PointCloud.cc

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

#758 Reintroduce pointcloud render without Float_V message (backport #755) has been created for branch gz-gui9
#759 Reintroduce pointcloud render without Float_V message (backport #755) has been created for branch ign-gui6 but encountered conflicts

Cherry-pick of d8b6a54 has failed:

On branch mergify/bp/ign-gui6/pr-755
Your branch is up to date with 'origin/ign-gui6'.

You are currently cherry-picking commit d8b6a54.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   src/plugins/point_cloud/PointCloud.cc

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

🦟 Bug fix [This bug](#494 (comment)) ## Summary Re-introduce point cloud rendering without the Float_V message. <img width="1198" height="996" alt="Screenshot From 2026-03-17 11-49-46" src="https://github.com/user-attachments/assets/fe538b5b-d93a-429b-9900-3b13345dece1" /> I tested it with the following program: ```C++ #include <iostream> #include <cmath> #include <thread> #include <chrono> #include <gz/msgs/pointcloud_packed.pb.h> #include <gz/msgs/PointCloudPackedUtils.hh> #include <gz/transport/Node.hh> int main(int argc, char** argv) { // Create a transport node. gz::transport::Node node; // Define the topic to publish on. std::string topic = "/sphere_points"; // Create a publisher for PointCloudPacked messages. auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic); if (!pub) { std::cerr << "Error advertising topic [" << topic << "]" << std::endl; return -1; } // Sphere parameters double radius = 1.0; int num_theta = 20; int num_phi = 40; // Prepare the message gz::msgs::PointCloudPacked msg; // Initialize header msg.mutable_header()->mutable_stamp()->set_sec(0); msg.mutable_header()->mutable_stamp()->set_nsec(0); // Initialize PointCloudPacked gz::msgs::InitPointCloudPacked(msg, "frame_id", false, {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}}); int num_points = (num_theta + 1) * num_phi; msg.set_width(num_points); msg.set_height(1); msg.set_is_dense(true); msg.mutable_data()->resize(num_points * msg.point_step()); gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x"); gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y"); gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z"); const double PI = std::acos(-1.0); for (int i = 0; i <= num_theta; ++i) { double theta = PI * i / num_theta; for (int j = 0; j < num_phi; ++j) { double phi = 2.0 * PI * j / num_phi; double x = radius * std::sin(theta) * std::cos(phi); double y = radius * std::sin(theta) * std::sin(phi); double z = radius * std::cos(theta); *iterX = static_cast<float>(x); *iterY = static_cast<float>(y); *iterZ = static_cast<float>(z); ++iterX; ++iterY; ++iterZ; } } std::cout << "Publishing a sphere with " << num_points << " points on topic [" << topic << "]" << std::endl; // Publish the message in a loop while (true) { if (!pub.Publish(msg)) { std::cerr << "Failed to publish message" << std::endl; } std::this_thread::sleep_for(std::chrono::milliseconds(1000)); } return 0; } ``` TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in `gz-math`. For now there is rudimentary support for colors in this PR. --------- Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com> Co-authored-by: Ian Chen <ichen@openrobotics.org> Co-authored-by: Ian Chen <iche@google.com> (cherry picked from commit d8b6a54)

🦟 Bug fix [This bug](#494 (comment)) ## Summary Re-introduce point cloud rendering without the Float_V message. <img width="1198" height="996" alt="Screenshot From 2026-03-17 11-49-46" src="https://github.com/user-attachments/assets/fe538b5b-d93a-429b-9900-3b13345dece1" /> I tested it with the following program: ```C++ #include <iostream> #include <cmath> #include <thread> #include <chrono> #include <gz/msgs/pointcloud_packed.pb.h> #include <gz/msgs/PointCloudPackedUtils.hh> #include <gz/transport/Node.hh> int main(int argc, char** argv) { // Create a transport node. gz::transport::Node node; // Define the topic to publish on. std::string topic = "/sphere_points"; // Create a publisher for PointCloudPacked messages. auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic); if (!pub) { std::cerr << "Error advertising topic [" << topic << "]" << std::endl; return -1; } // Sphere parameters double radius = 1.0; int num_theta = 20; int num_phi = 40; // Prepare the message gz::msgs::PointCloudPacked msg; // Initialize header msg.mutable_header()->mutable_stamp()->set_sec(0); msg.mutable_header()->mutable_stamp()->set_nsec(0); // Initialize PointCloudPacked gz::msgs::InitPointCloudPacked(msg, "frame_id", false, {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}}); int num_points = (num_theta + 1) * num_phi; msg.set_width(num_points); msg.set_height(1); msg.set_is_dense(true); msg.mutable_data()->resize(num_points * msg.point_step()); gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x"); gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y"); gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z"); const double PI = std::acos(-1.0); for (int i = 0; i <= num_theta; ++i) { double theta = PI * i / num_theta; for (int j = 0; j < num_phi; ++j) { double phi = 2.0 * PI * j / num_phi; double x = radius * std::sin(theta) * std::cos(phi); double y = radius * std::sin(theta) * std::sin(phi); double z = radius * std::cos(theta); *iterX = static_cast<float>(x); *iterY = static_cast<float>(y); *iterZ = static_cast<float>(z); ++iterX; ++iterY; ++iterZ; } } std::cout << "Publishing a sphere with " << num_points << " points on topic [" << topic << "]" << std::endl; // Publish the message in a loop while (true) { if (!pub.Publish(msg)) { std::cerr << "Failed to publish message" << std::endl; } std::this_thread::sleep_for(std::chrono::milliseconds(1000)); } return 0; } ``` TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in `gz-math`. For now there is rudimentary support for colors in this PR. --------- Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com> Co-authored-by: Ian Chen <ichen@openrobotics.org> Co-authored-by: Ian Chen <iche@google.com> (cherry picked from commit d8b6a54) # Conflicts: # src/plugins/point_cloud/PointCloud.cc

🦟 Bug fix [This bug](#494 (comment)) ## Summary Re-introduce point cloud rendering without the Float_V message. <img width="1198" height="996" alt="Screenshot From 2026-03-17 11-49-46" src="https://github.com/user-attachments/assets/fe538b5b-d93a-429b-9900-3b13345dece1" /> I tested it with the following program: ```C++ #include <iostream> #include <cmath> #include <thread> #include <chrono> #include <gz/msgs/pointcloud_packed.pb.h> #include <gz/msgs/PointCloudPackedUtils.hh> #include <gz/transport/Node.hh> int main(int argc, char** argv) { // Create a transport node. gz::transport::Node node; // Define the topic to publish on. std::string topic = "/sphere_points"; // Create a publisher for PointCloudPacked messages. auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic); if (!pub) { std::cerr << "Error advertising topic [" << topic << "]" << std::endl; return -1; } // Sphere parameters double radius = 1.0; int num_theta = 20; int num_phi = 40; // Prepare the message gz::msgs::PointCloudPacked msg; // Initialize header msg.mutable_header()->mutable_stamp()->set_sec(0); msg.mutable_header()->mutable_stamp()->set_nsec(0); // Initialize PointCloudPacked gz::msgs::InitPointCloudPacked(msg, "frame_id", false, {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}}); int num_points = (num_theta + 1) * num_phi; msg.set_width(num_points); msg.set_height(1); msg.set_is_dense(true); msg.mutable_data()->resize(num_points * msg.point_step()); gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x"); gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y"); gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z"); const double PI = std::acos(-1.0); for (int i = 0; i <= num_theta; ++i) { double theta = PI * i / num_theta; for (int j = 0; j < num_phi; ++j) { double phi = 2.0 * PI * j / num_phi; double x = radius * std::sin(theta) * std::cos(phi); double y = radius * std::sin(theta) * std::sin(phi); double z = radius * std::cos(theta); *iterX = static_cast<float>(x); *iterY = static_cast<float>(y); *iterZ = static_cast<float>(z); ++iterX; ++iterY; ++iterZ; } } std::cout << "Publishing a sphere with " << num_points << " points on topic [" << topic << "]" << std::endl; // Publish the message in a loop while (true) { if (!pub.Publish(msg)) { std::cerr << "Failed to publish message" << std::endl; } std::this_thread::sleep_for(std::chrono::milliseconds(1000)); } return 0; } ``` TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in `gz-math`. For now there is rudimentary support for colors in this PR. --------- Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com> Co-authored-by: Ian Chen <ichen@openrobotics.org> Co-authored-by: Ian Chen <iche@google.com> (cherry picked from commit d8b6a54)

🦟 Bug fix [This bug](#494 (comment)) ## Summary Re-introduce point cloud rendering without the Float_V message. <img width="1198" height="996" alt="Screenshot From 2026-03-17 11-49-46" src="https://github.com/user-attachments/assets/fe538b5b-d93a-429b-9900-3b13345dece1" /> I tested it with the following program: ```C++ #include <iostream> #include <cmath> #include <thread> #include <chrono> #include <gz/msgs/pointcloud_packed.pb.h> #include <gz/msgs/PointCloudPackedUtils.hh> #include <gz/transport/Node.hh> int main(int argc, char** argv) { // Create a transport node. gz::transport::Node node; // Define the topic to publish on. std::string topic = "/sphere_points"; // Create a publisher for PointCloudPacked messages. auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic); if (!pub) { std::cerr << "Error advertising topic [" << topic << "]" << std::endl; return -1; } // Sphere parameters double radius = 1.0; int num_theta = 20; int num_phi = 40; // Prepare the message gz::msgs::PointCloudPacked msg; // Initialize header msg.mutable_header()->mutable_stamp()->set_sec(0); msg.mutable_header()->mutable_stamp()->set_nsec(0); // Initialize PointCloudPacked gz::msgs::InitPointCloudPacked(msg, "frame_id", false, {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}}); int num_points = (num_theta + 1) * num_phi; msg.set_width(num_points); msg.set_height(1); msg.set_is_dense(true); msg.mutable_data()->resize(num_points * msg.point_step()); gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x"); gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y"); gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z"); const double PI = std::acos(-1.0); for (int i = 0; i <= num_theta; ++i) { double theta = PI * i / num_theta; for (int j = 0; j < num_phi; ++j) { double phi = 2.0 * PI * j / num_phi; double x = radius * std::sin(theta) * std::cos(phi); double y = radius * std::sin(theta) * std::sin(phi); double z = radius * std::cos(theta); *iterX = static_cast<float>(x); *iterY = static_cast<float>(y); *iterZ = static_cast<float>(z); ++iterX; ++iterY; ++iterZ; } } std::cout << "Publishing a sphere with " << num_points << " points on topic [" << topic << "]" << std::endl; // Publish the message in a loop while (true) { if (!pub.Publish(msg)) { std::cerr << "Failed to publish message" << std::endl; } std::this_thread::sleep_for(std::chrono::milliseconds(1000)); } return 0; } ``` TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in `gz-math`. For now there is rudimentary support for colors in this PR. --------- Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com> Co-authored-by: Ian Chen <ichen@openrobotics.org> Co-authored-by: Ian Chen <iche@google.com> (cherry picked from commit d8b6a54) # Conflicts: # src/plugins/point_cloud/PointCloud.cc

🦟 Bug fix [This bug](#494 (comment)) ## Summary Re-introduce point cloud rendering without the Float_V message. <img width="1198" height="996" alt="Screenshot From 2026-03-17 11-49-46" src="https://github.com/user-attachments/assets/fe538b5b-d93a-429b-9900-3b13345dece1" /> I tested it with the following program: ```C++ #include <iostream> #include <cmath> #include <thread> #include <chrono> #include <gz/msgs/pointcloud_packed.pb.h> #include <gz/msgs/PointCloudPackedUtils.hh> #include <gz/transport/Node.hh> int main(int argc, char** argv) { // Create a transport node. gz::transport::Node node; // Define the topic to publish on. std::string topic = "/sphere_points"; // Create a publisher for PointCloudPacked messages. auto pub = node.Advertise<gz::msgs::PointCloudPacked>(topic); if (!pub) { std::cerr << "Error advertising topic [" << topic << "]" << std::endl; return -1; } // Sphere parameters double radius = 1.0; int num_theta = 20; int num_phi = 40; // Prepare the message gz::msgs::PointCloudPacked msg; // Initialize header msg.mutable_header()->mutable_stamp()->set_sec(0); msg.mutable_header()->mutable_stamp()->set_nsec(0); // Initialize PointCloudPacked gz::msgs::InitPointCloudPacked(msg, "frame_id", false, {{"xyz", gz::msgs::PointCloudPacked::Field::FLOAT32}}); int num_points = (num_theta + 1) * num_phi; msg.set_width(num_points); msg.set_height(1); msg.set_is_dense(true); msg.mutable_data()->resize(num_points * msg.point_step()); gz::msgs::PointCloudPackedIterator<float> iterX(msg, "x"); gz::msgs::PointCloudPackedIterator<float> iterY(msg, "y"); gz::msgs::PointCloudPackedIterator<float> iterZ(msg, "z"); const double PI = std::acos(-1.0); for (int i = 0; i <= num_theta; ++i) { double theta = PI * i / num_theta; for (int j = 0; j < num_phi; ++j) { double phi = 2.0 * PI * j / num_phi; double x = radius * std::sin(theta) * std::cos(phi); double y = radius * std::sin(theta) * std::sin(phi); double z = radius * std::cos(theta); *iterX = static_cast<float>(x); *iterY = static_cast<float>(y); *iterZ = static_cast<float>(z); ++iterX; ++iterY; ++iterZ; } } std::cout << "Publishing a sphere with " << num_points << " points on topic [" << topic << "]" << std::endl; // Publish the message in a loop while (true) { if (!pub.Publish(msg)) { std::cerr << "Failed to publish message" << std::endl; } std::this_thread::sleep_for(std::chrono::milliseconds(1000)); } return 0; } ``` TODO: Color support: I think we should not implement the color rendering support here, rather perhaps we should have some PointCloud representation in `gz-math`. For now there is rudimentary support for colors in this PR. --------- (cherry picked from commit d8b6a54) Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com> Co-authored-by: Arjo Chakravarty <arjoc@intrinsic.ai> Co-authored-by: Ian Chen <ichen@openrobotics.org> Co-authored-by: Ian Chen <iche@google.com>

arjo129 added 3 commits March 4, 2026 09:34

Attempt to display point clouds without topic

40f1409

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

Merge remote-tracking branch 'origin/main' into arjo/fix/reintroduce_…

e87f8a1

…pointcloud_render

PointCloud: Use minimum color and add warning when color fields are m…

32a391b

…issing Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

arjo129 requested a review from jennuine as a code owner March 17, 2026 03:29

arjo129 requested a review from azeey March 17, 2026 03:29

azeey added this to Core development Mar 17, 2026

github-project-automation bot moved this to Inbox in Core development Mar 17, 2026

github-actions bot added the 🏯 kura label Mar 17, 2026

arjo129 added 3 commits March 17, 2026 03:31

spelling

60b393a

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

style

3372839

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

style

4295c81

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

arjo129 changed the title ~~Arjo/fix/reintroduce pointcloud render~~ Reintroduce pointcloud render without Float_V message Mar 17, 2026

iche033 reviewed Mar 17, 2026

View reviewed changes

arjo129 and others added 2 commits March 18, 2026 03:15

Update src/plugins/point_cloud/PointCloud.cc

7927f77

Co-authored-by: Ian Chen <ichen@openrobotics.org> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com>

Apply suggestions from code review

6734949

Co-authored-by: Ian Chen <iche@google.com> Signed-off-by: Arjo Chakravarty <arjo129@gmail.com>

arjo129 requested a review from iche033 March 18, 2026 02:15

iche033 approved these changes Mar 18, 2026

View reviewed changes

github-project-automation bot moved this from Inbox to In review in Core development Mar 18, 2026

arjo129 merged commit d8b6a54 into main Mar 18, 2026
8 checks passed

arjo129 deleted the arjo/fix/reintroduce_pointcloud_render branch March 18, 2026 20:16

github-project-automation bot moved this from In review to Done in Core development Mar 18, 2026

mergify bot mentioned this pull request Mar 18, 2026

Reintroduce pointcloud render without Float_V message (backport #755) #756

Merged

11 tasks

mergify bot mentioned this pull request Mar 18, 2026

Reintroduce pointcloud render without Float_V message (backport #755) #757

Open

11 tasks

mergify bot mentioned this pull request Mar 18, 2026

Reintroduce pointcloud render without Float_V message (backport #755) #758

Merged

11 tasks

mergify bot mentioned this pull request Mar 18, 2026

Reintroduce pointcloud render without Float_V message (backport #755) #759

Open

11 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Reintroduce pointcloud render without Float_V message#755

Reintroduce pointcloud render without Float_V message#755
arjo129 merged 8 commits intomainfrom
arjo/fix/reintroduce_pointcloud_render

arjo129 commented Mar 17, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

arjo129 commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026 •

edited

Loading

Uh oh!

azeey commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026

Uh oh!

azeey commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

arjo129 commented Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦟 Bug fix

Summary

Checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

arjo129 commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ No backport have been created

Uh oh!

azeey commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026

❌ No backport have been created

Uh oh!

azeey commented Mar 18, 2026

Uh oh!

mergify bot commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Backports have been created

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

arjo129 commented Mar 17, 2026 •

edited

Loading

mergify bot commented Mar 18, 2026 •

edited

Loading

mergify bot commented Mar 18, 2026 •

edited

Loading