Skip to content
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

Missing parenthesis in loss #3

Open
ctiennot opened this issue Jun 19, 2017 · 2 comments
Open

Missing parenthesis in loss #3

ctiennot opened this issue Jun 19, 2017 · 2 comments

Comments

@ctiennot
Copy link

Hi again,

In the file model.py at line 46, shouldn't:
cost = tf.reduce_mean(sum_square_d / 55.0*74.0 - 0.5*sqare_sum_d / math.pow(55*74, 2))

be replaced by
cost = tf.reduce_mean(sum_square_d / (55.0*74.0) - 0.5*sqare_sum_d / math.pow(55*74, 2))

to match equation (4) form the paper? (I think you just forgot the parenthesis).

Best,

Clement

@drstarson
Copy link

@ctiennot Did you succeed training the model and getting the results after adding the parenthesis. It looks like the parenthesis are needed, but after adding them, the system doesn't seem to be training well. I'm getting low loss (~0.2), but the resulting depth images are all black. Would appreciate any advice or additional info.

@josslynZcn
Copy link

@drstarson i have experience the same problem with you. and i wonder if you solve this problem .would appreciate with the solution about this problen. i would be so thankful if you connect with me zhangchenni121@126.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants