glow icon indicating copy to clipboard operation
glow copied to clipboard

Is a bug in calling "get_variable_ddi" causing NaN values?

Open omidsakhi opened this issue 6 years ago • 2 comments

The function "get_variable_ddi" at https://github.com/openai/glow/blob/master/tfops.py#L58 has an init=False parameter. There are currently four calls to this function and I don't see the init to be True in any of those calls which means that the initialization never gets assigned. Am I missing something here?

omidsakhi avatar Aug 20 '18 19:08 omidsakhi

init=True is set here: https://github.com/openai/glow/blob/master/model.py#L72

dpkingma avatar Aug 22 '18 20:08 dpkingma

Thanks. I missed that one.

omidsakhi avatar Aug 22 '18 20:08 omidsakhi