Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD] vgg16 tils model #112

Merged
merged 10 commits into from
Apr 7, 2023
Merged

[ADD] vgg16 tils model #112

merged 10 commits into from
Apr 7, 2023

Conversation

kaczmarj
Copy link
Member

@kaczmarj kaczmarj commented Apr 7, 2023

this pr adds the vgg16 tils model from https://github.com/ShahiraAbousamra/til_classification

the original model was implemented in tensorflow slim and the weights were converted to a pytorch state dict using the script https://github.com/SBU-BMI/wsinfer/blob/main/scripts/convert_tf_to_pytorch_til_vgg16.py

the tensorflow model file was downloaded from Box (https://stonybrookmedicine.app.box.com/file/905508899913?s=r0p2h2oh53zw52o6fm6qrqjtcwd3p0ga). the weights files were those with the prefix tcga_vgg16_mix_new3.ckpt.

i am not 100% sure that the image normalization is correct. at the moment, images are normalized to range [-1, 1]. this appears to give correct output. normalizing each image to mean 0 and variance 1 gave poor results so it is probably not correct. using original image values [0, 255] also gave poor results. the original code does reference normalizing to [-1, 1] but it is not fully clear that this was used for this model.

fixes #110

@kaczmarj kaczmarj merged commit f92d657 into main Apr 7, 2023
@kaczmarj kaczmarj deleted the add/vgg16-tils branch April 7, 2023 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add TILs VGG16 model
1 participant