Kitware
3dtiles pointcloud and mesh processing
Note the number of commits is misleading as I moved [email protected]:core3d/danesfield-conda-recipes.git to the conda-recipes directory including history.
@danlipsa I have small changes to Dockerfile, tools/run_danesfield.py, and the VTK recipe that I need to merge. Otherwise this looks good
@danlipsa overall looks fine to me, I would recommend changing out how the SSH stuff is done in the Dockerfile though. Also, regarding the number of commits, if you did something like a subtree add, you can usually add a --squash flag to have it combine all commits into a single one. Probably not worth re-doing it in this case, but I would recommend squashing this PR on merge (so that we don't drown out the history of the main project).
@dmjoy I agree, we should squash. I don't think the recipe's history is very important.
@jacobderosa I changed danesfield-conda-recipes repo -> conda-recipes dir to have only one commit. Diff seems to show nothing changed. Please review and merge.
Not sure if you addressed the suggestion from @dmjoy regarding SSH of if you not doing that at all now. That seems that something we should do.
@danlipsa I'll add that along with my own changes after this is all merged. For now this looks good to me
I have a few more commits I added when I worked on when added offset inside GLTF. I'll rebase and open another pull request.