-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Apply Transformation Globally and save as PCD #1979
Comments
I think you are misunderstanding something. both of your method should result in the same cloud, except if the transformation larger than 10^4. because if it is larger it could corrupt your points which was not applied to global. The only cause to have that option is to prevent users to loss their point's precision because of big transformations. what you really want to achieve? |
Ok I then I might have to refrase something here: |
For example I did a rotation of the pointcloud by 1.5 deg in Rx and saved the resulting point cloud as pcd. |
Ok, so I took the time to look at the issue this week-end, and here are my observations:
So there were 2 potential reasons why the PCD files were the same (but mostly because the Global Shift & Scale information was dropped...). Can you retry with the 2.13.1.preview version here: https://www.cloudcompare.org/release/CloudCompare_v2.13.1.preview_setup_x64.exe |
I also looked at it, and found out something was strange, that is why i did not react to the last message. (related to global shift was not stored or stored somewhat badly) open a cloud,apply large transofrmation, save as pcd rearead transformed pcd, and the two were different. about a half a meter difference. seems to be fixed with the preview. |
Can you share your PCD file with us? (or you can send it to admin@cloudcompare.org) Something that was not managed in the old versions is the concept of sensors. This is now the case. That's the only difference I see (well, apart that the first support of PCD files was done manually, by Luca Penasa, and we have now transitioned to using the real PCL library to import and export PCD files). |
Also, for the example above, are you applying the rotation to the local or to the global coordinate system (it only makes sense if you have a Global Shift defined) |
You got mail ;) I described again what transformation I did in CC. |
I always get an email delivery error when I try to sent it to the mail you mentioned, here you go: cut.pcd is the original one (unrotated, just cut from some data in CC.2.9.1 and saved again) cut_rotated_saved_2.13.1_preview.pcd is the one loaded, rotated and saved again in CC2.13.1_preview cut_rotated_saved_2.9.1.pcd ist the one loaded, rotated and saved in CC.2.9.1. The rotation I did was: I marked the cloud, |
Ok, so if you rotate the PCD file with 2.13.1, save it, and reopen it with 2.13.1, it works. I didn't pay attention to the fact that you were rotating in version 2.9 (which is more than 6 years old 😅). As I said, this version was probably not using the PCL library to save the PCD file, and it was ignoring the 'sensor' when applying the transformation. |
You could try to remove the sensor below the cloud in v2.9 before saving the PCD file, that should help. |
Describe the bug
If I use the "Apply to global coordinates" feature in the transform dialoge (in Axis, Angle) it seems like it is still not applied globally, but instead still changes something different.
If I then save the result as a .pcd the result is identical, no matter if I enable or disable this option.
Also: If I save the cloud as .txt and reload it, and then save as new .pcd the result is different.
How to reproduce
Both PCDs are now the same, but they shouldn't be. Also: The resulting .pcd is always different than one saved with CC Version 2.12.X or 2.9.1, no matter if I enable or disable the option.
Expected behaviour
If that option is now there: (why is it needed anyways?)
Additional context
Workaround:
When you transform, but then save e.g. as .csv/.txt, then import again and save as .pcd the Problem is not occuring.
Your environment
The text was updated successfully, but these errors were encountered: