carpedm20 / carpedm20/DCGAN-tensorflow

why use the sigmoid?

Open
#222 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
7.2k
Forks
2.6k
PR merge metrics
No merged PRs in 30d

Description

Hello man,
in model.py
line 111 `self.D_, self.D_logits_ = self.discriminator(self.G, self.y, reuse=True)`
line 311
pastedgraphic-3
d_loss
pastedgraphic-2
is it a bug ?
may be it should be
sigmoid_cross_entropy_with_logits(self.D_logits, tf.ones_like(self.D_logits)))
and we need not to calculate sigmoid(h4).

i am a noob . if i am wrong, i hope that you get tell me why ?
thanks.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading model.py around lines 111 and 311, then trace the discriminator output and d_loss formulation. Compare the sigmoid output with the logits passed to TensorFlow's cross-entropy operation; done means confirming whether this is a bug and documenting the explanation or required change.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, tensorflow
Domain
machine-learning
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.