tensorflow / tensorflow/models

The real_nvp example code seems wrong in computing log_det from the decoder function

Open
#7,465 0 comments 0 reactions 1 assignee View on GitHub

@jaeyounkim is already working on this.

Since Jun 22, 2020.

models:research type:support
Dominant language
Python
Stars
77.7k
Forks
44.8k
PR merge metrics
No merged PRs in 30d

Description

#7177 # System information

  • What is the top-level directory of the model you are using: research/real_nvp
  • Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no
  • OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04 (in docker)
  • TensorFlow installed from (source or binary): Binary
  • TensorFlow version (use command below): 1.14.0
  • Bazel version (if compiling from source):
  • CUDA/cuDNN version: 9.0.176
  • GPU model and memory: GTX 1080 Ti
  • Exact command to reproduce:
Describe the problem

Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.

In real_nvp_multiscale_dataset.py, at L806 and L818, you mistakenly override log_diff_1 and log_diff. The outputs are supposed to be assigned to inc_log_diff and then add to log_diff_1 or log_diff respectively, as is done in the encoder at L744 and L756.

This problem have not affected the results of your example code, since the log_diff output from the decoder is never used. However, I think it might be better fixed, since it is an example code from the TensorFlow models repo.

Source code / logs

Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.