@expenses @CharlesWardlaw I haven't had a chance to look at this PR yet, but please be aware there is already work in progress on Point Cloud import:
I merged this manually in order to fix a conflict and am closing this PR.
@CharlesWardlaw I think this pull request should merge into the blender-v4.0-release
branch and not into main
.
These changes look good and work well in my tests. Thank you for the fix!
Hi @devin-bayly. Nice progress!
Ok, so integrating that code from above I have the color being brought in. I'm curious what the correct name would be to have blender automatically pick up…
If we want this as a bug fix for 4.0, then this pull request should be changed to merge into blender-v4.0-release
rather than into main
. Thanks!
If you don't mind, can you please run 'make format' to fix the formatting, please?
Otherwise, apart from the minor comment regarding adding a leading slash to the default root path, this looks…
Maybe the default string should be "/root", so we don't have to prepend the leading slash each time?
Also, I think USD I/O code is currently fairly abusing the Report system... Lots of these messages should rather be processed through the logging system imho (se are using CLOG), reports should…
Thank you so much for this important fix!
The USD implementation looks good to me and works in my test. Since you mentioned that the USD related changes will likely be submitted separately, I…
@makowalski would love to have your PoV on USD-related changes in this code.
I will aim to finish my review tomorrow, but so far the changes look good!