-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve unet_simple #18
Comments
What did the original paper propose? |
Hi, In the original paper, the authors used 23 convolutional layers. I think these may be too many for some datasets and it would be better to provide a way to customize the number of layers in the model to avoid overfitting. The problem with The original architecture from the paper: |
I think you might want to add an argument to the function for the conv depth. |
Yes, this is in line what I had in mind. I was also planning to rewrite the whole |
I will try to look into it soon. If I manage to build something, I will let you know. No need to assign me for now. |
Assign it to me, I'll be finished soon. |
Description
I would like to improve the initialization of
unet_simple
especially regarding making it similar to what the original paper proposed.Similar Features
It is currently implemented in
unet_simple
Feature Details
Improve upsampling
Review Downsampling
Proposed Implementation
None yet
Thank you
The text was updated successfully, but these errors were encountered: