Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

PGGAN blending issue #98

Open
GongXinyuu opened this issue Jan 12, 2020 · 2 comments
Open

PGGAN blending issue #98

GongXinyuu opened this issue Jan 12, 2020 · 2 comments

Comments

@GongXinyuu
Copy link

mergeLayer = self.alpha > 0 and len(self.scaleLayers) > 1

Should this line be mergeLayer = self.alpha > 0 and len(self.fromRGBLayers) > 1?

As the previous line calculate y on the condition of

if self.alpha > 0 and len(self.fromRGBLayers) > 1:

@Molugan
Copy link
Contributor

Molugan commented Jan 13, 2020

You're right, the size of fromRGBLayers and scaleLayers are supposed to be the same at each step, but it's not super clean to refer to both in the code. I'll clean that up.

@LiUzHiAn
Copy link

LiUzHiAn commented Nov 15, 2021

Hi,

I don't think the size of fromRGBLayers and scaleLayers are the exactly same. Since the scale0 was not added to the scaleLayers.

self.groupScaleZero.append(EqualizedConv2d(dimEntryScale0, depthScale0,
3, padding=1,
equalized=equalizedlR,
initBiasToZero=initBiasToZero))
self.groupScaleZero.append(EqualizedLinear(depthScale0 * 16,
depthScale0,
equalized=equalizedlR,
initBiasToZero=initBiasToZero))

But the RGBLayer for the scale0 was added.

self.fromRGBLayers.append(EqualizedConv2d(dimInput, depthScale0, 1,
equalized=equalizedlR,
initBiasToZero=initBiasToZero))

So, mergeLayer = self.alpha > 0 and len(self.fromRGBLayers) > 1 may be more reasonable. Did I misunderstand?

Thanks for your work.

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

No branches or pull requests

3 participants