Compatility issue about pixel2meshplusplus HOT 9 CLOSED

Hi, @bharadwajdhornala. The main compatibility problem is that the version provided by tf_nndistance, namely Chamfer Loss, is the GPU version. If you want to use it on CPU, you need to use Chamfer Loss of CPU version. I will upload the Loss code of CPU version to the external/tf_nndistance_cpu.py, please import this library.

from pixel2meshplusplus.

Can you upload the tf_nndistance_cpu_so.so file so that I can continue my further execution. Or can you please guide for the demo.py execution using CPU.

from pixel2meshplusplus.

I got an error in the below
File "/home/bharadwaj/Pixel2MeshPlusPlus/external/tf_nndistance_cpu.py", line 17, in nn_distance_cpu
pc1_expand_tile = tf.tile(tf.expand_dims(pc1,2), [1,1,M,1])
TypeError: Failed to convert object of type <class 'list'> to Tensor. Contents: [1, 1, None, 1]. Consider casting elements to a supported type.

Please suggest.

from pixel2meshplusplus.

Hi @bharadwajdhornala , tf_nndistance_cpu.py do not require compile. I think and the error may related to unknown tensor shape for placeholder, so I have modified the code of cpu version chamfer loss. For now, you do not need to expand dimension of batch_size before calling loss function, and it also should fix the bug of None tensor size.

Do let me know if you have any other questions.

from pixel2meshplusplus.

Thanks @walsvid , When I run the code, I got this error
pc1_expand_tile = tf.tile(tf.expand_dims(pc1,2), [1,1,M,1])
ValueError: Shape must be rank 5 but is rank 4 for 'Tile' (op: 'Tile') with input shapes: [1,1,1,?,3], [4]. So, I have tried to increase rank 4 to 5,and it worked, it gave an obj file. But when I run it another time , its getting Killed. If I want to replace plane images with any other images, should I change the cameras.txt also?

from pixel2meshplusplus.

That may related to unnecessary expan dims, may be after compute the CPU version chamfer, you should squeeze or reshape the tensor to remove the unnecessary dimensions.

Yes, if the images changed the camera parameter should also be changed. All the camera data are provided for shapenet.

from pixel2meshplusplus.

Can you please upload the changes to be made in chamfer for reshaping tensor.

from pixel2meshplusplus.

I mean you should check your code for extra expand dimension operations to make sure the output is correct. At present, we can only guarantee that the GPU version is definitely correct. Whether there is bugs with the CPU version, you need to check it by yourself first. I will check it later when I have time.

from pixel2meshplusplus.

Thanks @walsvid for your valuable suggestions.

from pixel2meshplusplus.