Skip to content
This repository was archived by the owner on Feb 7, 2025. It is now read-only.

Add 3d percetual loss#158

Merged
marksgraham merged 6 commits into
mainfrom
10-add-3d-feature-loss
Jan 4, 2023
Merged

Add 3d percetual loss#158
marksgraham merged 6 commits into
mainfrom
10-add-3d-feature-loss

Conversation

@Warvito

@Warvito Warvito commented Dec 27, 2022

Copy link
Copy Markdown
Collaborator

Implements #10

Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>

@marksgraham marksgraham left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Walter - looks good to me.

A couple of changes needed to get the tests running on my system

Comment thread generative/losses/perceptual.py
Comment thread generative/losses/perceptual.py Outdated
Comment thread generative/losses/perceptual.py Outdated
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito Warvito requested a review from marksgraham January 4, 2023 16:21
@Warvito

Warvito commented Jan 4, 2023

Copy link
Copy Markdown
Collaborator Author

Thanks for the review @marksgraham . Could you please check if the changes look okay?

@marksgraham marksgraham left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good now!

@marksgraham marksgraham merged commit 0698e4e into main Jan 4, 2023
@marksgraham marksgraham deleted the 10-add-3d-feature-loss branch January 4, 2023 21:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants