model.sensing.image_to_points color bug

krishauser / Klampt

Kris' Locomotion and Manipulation Planning Toolkit

BSD 3-Clause "New" or "Revised" License

377 stars 96 forks source link

model.sensing.image_to_points color bug #119

Closed patricknaughton01 closed 3 years ago

patricknaughton01 commented 3 years ago

I was testing the 0.9 branch and I think this line https://github.com/krishauser/Klampt/blob/37cbcf2df7af496fca5a02dc43af3f39a926ec96/Python/klampt/model/sensing.py#L369

needs to concatenate along dim 1 instead of 2 because at this point pts and color have been reshaped to only have 2 dimensions. As is, was giving me an error, making this change locally fixed it.

krishauser commented 3 years ago

Ah yes, thanks, can you make a PR?

patricknaughton01 commented 3 years ago

Closed by #120