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

Add RadImageNet to Perceptual Loss#153

Merged
Warvito merged 4 commits into
mainfrom
26-add-radimagenet-option-to-perceptual-loss
Jan 8, 2023
Merged

Add RadImageNet to Perceptual Loss#153
Warvito merged 4 commits into
mainfrom
26-add-radimagenet-option-to-perceptual-loss

Conversation

@Warvito

@Warvito Warvito commented Dec 21, 2022

Copy link
Copy Markdown
Collaborator

Implements #26

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

Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito Warvito linked an issue Dec 21, 2022 that may be closed by this pull request
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito Warvito marked this pull request as ready for review December 22, 2022 13:54
@Warvito

Warvito commented Dec 22, 2022

Copy link
Copy Markdown
Collaborator Author

I am developing the other entrypoints and converting the networks from https://github.com/BMEII-AI/RadImageNet to pytorch (working with Zelong Liu for this). The plan is to add support to at least inceptionV3 and mobilenet.

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

@danieltudosiu danieltudosiu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall I would approve the pull request. But there are some quality of life improvements.

Comment thread generative/losses/perceptual.py
Comment thread generative/losses/perceptual.py
Comment thread generative/losses/perceptual.py
@Warvito Warvito merged commit e9597bb into main Jan 8, 2023
@Warvito Warvito deleted the 26-add-radimagenet-option-to-perceptual-loss branch January 8, 2023 12:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants