-
Notifications
You must be signed in to change notification settings - Fork 46
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
Issues with up_ratio #33
Comments
I'm also trying for a different up_ratio. Were you able to do it? |
Yes. I got it to work.
I trained multiple versions of PU-GAN with different upsampling ratios. During testing, I would edit the lines 22, 23, and 24 in I also found another mistake during the testing phase. It is in line 262, 263, and 264. The code is measuring the output points for the first point and is then assuming that all the other point clouds being fed in are the same size. So if your first point cloud was 2k point with 4x upsampling. The testing would output 8k points. Even when you feed it 8k point cloud input, the output would be 8k points. So you should find the number of output points within the loop.
I am not sure what other changes I made to the code since it has been a long time. Feel free to send me an email and I can share my code (a bit messy). We can discuss this work further. |
Hey !! ty for the response! I tried to do what you've done but i still got this error:
Its because the h5 file. Since isto not 4256 =1024 (the one that is available) and is 10256=2560 i got this error? Did you changed something on the data_loader too? |
Just comparing the two files, I can't see any changes.
So basically randomly choosing a single point on each point cloud to extract a single patch. I believe I had 24k point clouds. so ended up with 24k patches. |
Thank you so much!!! |
I am trying to trian the model for up_ratio=8 but I realized there are a lot of error in the code that doesn't make it possible to train for that up_ratio.
E.g.:
in model.py, line 34:
self.input_y = tf.placeholder(tf.float32, shape=[self.opts.batch_size, int(4*self.opts.num_point),3])
Maybe this should be:
self.input_y = tf.placeholder(tf.float32, shape=[self.opts.batch_size, int(self.opts.up_ratio*self.opts.num_point),3])
Similarly, in model.py, line 253:
This second issue I do not understand. What is the purpose of this line? Should this not be here?
Also found this issue in data_loader.py, line 36:
num_4X_point = int(opts.num_point*4)
Any help would be appreciated.
Thanks
The text was updated successfully, but these errors were encountered: