Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

Some tweaks #91

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

Some tweaks #91

wants to merge 25 commits into from

Conversation

kasai2210
Copy link

Hi!
I am an undergrad and was trying to implement the Voxelnet paper while I came across your repository. I truly appreciate your work and the effort you put in while implementing the paper. But there were some tweaks I had to do make the code train on the dataset. I wanted to include them in your repository(main) so that if anyone else tries to directly implement it then they don't face the same issues. I have just started learning machine learning literature and please tell me if you find anything wrong or unnecessary in the pull request I am making. Thanks.

kasai2210 and others added 25 commits January 11, 2021 15:35
-> Changed import statement for imread.
-> Updated directories
->  Changed the output_name initialisation statement to actually write-over the existing file
changed 
tf.app.run(main) 
to 
tf.compat.v1.app.run(main)
as app.run has been shifted in v2
made compatible with tf version 1 by replacing tf. with tf.compat.v1. wherever possible
made compatible with tf.version 1
made compatible with tf.version1
changed kip_dims to keepdims in line 66
made compatible with tf v1
Made compatible with tf v1
made compatible with tf v1
Changed batch size in line 29, from 2 to 1.
Added comment explaining argparse functions
Added links for understanding tf.Graph(), tf.sess()
Allows for splitting raw Kitti data into training and validation sets
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants