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

DenseNet FCN #131

Closed
FuriouslyCurious opened this issue Mar 28, 2017 · 12 comments
Closed

DenseNet FCN #131

FuriouslyCurious opened this issue Mar 28, 2017 · 12 comments

Comments

@FuriouslyCurious
Copy link
Contributor

Hi @orashi and @gpleiss - thank you for adding DenseNet!

I was wondering if you have looked at DenseNet Fully Convolutional paper which gives phenomenal scores for segmentation : https://arxiv.org/pdf/1611.09326v1.pdf

I will owe you all a round of beers if you can consider implementing DenseNet FCN too.

Great day,
FC

@gpleiss
Copy link
Contributor

gpleiss commented Mar 29, 2017

Haha who can say no to that 😄

The original implementation is in Theano. We could probably use the saved weights for that model, but it may be better to train from scratch.

It might make sense to hold off until the CamVid dataset is added (#90).

@felixgwu
Copy link

felixgwu commented Apr 2, 2017

Hi, @FuriouslyCurious I have an implementation of FC-DenseNet at head already.
I can submit a PR is needed.
However, I couldn't match their reported mIoU and accuracy by training from scratch.
I've submitted an issue to their repo, but he is busy to help these days.

@FuriouslyCurious
Copy link
Contributor Author

Thank you @gpleiss and @felixgwu

I am planning to train a Dense FC model from scratch using medical data and publish the weights / trained model for medical research.

@felixgwu if you share the code in a GIST I will love to try it out for some experiments I am running.

@felixgwu
Copy link

felixgwu commented Apr 2, 2017

@FuriouslyCurious
Here is the code of the model. Hopefully, it helps your research.
https://gist.github.com/felixgwu/045c887b6ccdf0edf4648da0c40bcc12

@FuriouslyCurious
Copy link
Contributor Author

FuriouslyCurious commented Apr 2, 2017

Thank you @felixgwu !

@ycszen FYI: DenseNet FCN code in GIST above ^

@yu-changqian
Copy link

Thank you! @FuriouslyCurious

@youngfly11
Copy link

How can i import the saved weight in the original implementation in Theano to the pytorch with the same architechture? @felixgwu
As I know, the saved weight is numpy format. and it seem difficult to initial the weight in pytorch one by one, because the net is so deep. Thank you for advanced

@EliasVansteenkiste
Copy link

@felixgwu Do you remember how far you where off from the original reported mIoU and accuracy by training from scratch?

@EliasVansteenkiste
Copy link

I found your results in FC-DenseNet issue 11.
Did you manage to further improve the results?

@FuriouslyCurious
Copy link
Contributor Author

@EliasVansteenkiste and @felixgwu Check out the Keras Tiramisu implementation below: Developer @titu1994 used SubPixelConvolution instead of Deconvolution as default method for upsampling. Not sure if that helps accuracy, but worth trying.

https://github.com/titu1994/Fully-Connected-DenseNets-Semantic-Segmentation

@alykhantejani
Copy link
Contributor

If you want to use sub-pixel convolution for upscaling you can use the PixelShuffle layer in PyTorch

@youngfly11
Copy link

@felixgwu . Thank you for you code implementation of FC-DenseNet in pytorch. Recent, i want to reproduce the result about FC-DenseNet, but my loss does not converge at all. would you share me you dataLoader so that i can compare it with my code? and if i initialize the learning rate with 1e-3, the loss will be infinity. I am so puzzled with it .

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

No branches or pull requests

7 participants