tensorflow / tensorflow/models

[slim] Bug in replacing activation function in MobileNetV2

Open
#7,032 2 comments 0 reactions 2 assignees View on GitHub

@marksandler2 is already working on this.

Since Jun 25, 2020.

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

Description

  • What is the top-level directory of the model you are using: slim
  • Have I written custom code: no
  • OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
  • TensorFlow installed from (source or binary): binary
  • TensorFlow version (use command below): 1.10.
  • Bazel version (if compiling from source): -
  • CUDA/cuDNN version: CUDA 9
  • GPU model and memory: 1080Ti, 11GB
  • Exact command to reproduce: Training MobileNetV2 with activation function other than ReLU6

It seems to me that there's a bug in here, where you edit the conv defaults of MNv2 to be with other activation function. While writing the conv defaults to conv_defaults, the variable used later is conv_defs. This results in no change of the activation function.

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.