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

Update code and remove extra parts #64

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

FBehrad
Copy link

@FBehrad FBehrad commented Nov 1, 2021

There are three extra layers in this model (inp_res in green block and two blue blocks). As the winner of BraTS 2019 used the same architecture and explained it more clear, I have removed extra layers and updated the number of convolution filters. Also, I've upgraded the code and now it supports tensorflow version 2.

FBehrad and others added 5 commits November 1, 2021 10:36
According to the given structure in the paper, there is an extra convolutional layer in green block (inp_res). Also, before the final layer there is another extra layer called Input_Dec_Output. It is probably because the article is a little vague, but I checked this architecture with the paper of the winner of BraTS 2019. The winner of BraTS 2019 has used the same architecture, but they have explained it much clear. This is their paper : https://link.springer.com/chapter/10.1007/978-3-030-46640-4_22
Tensorflow has implemented group normalization. First install tensorflow addons using !pip install -q -U tensorflow-addons==0.9.1, then use it.
In order to support tensorflow 2, I've replaced some functions of keras with their equivalents in tensorflow.
As I checked the code with these versions and it works, I recommend to
use these versions: tensorflow(2.1.0), tensorflow addons (0.9.1) and
keras (2.2.4)
As we have upgraded green block in previous commits, it causes some errors. I compared the architecture with that of the winner of 2019, so I understood the number of filters in some layers should be changed.
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.

None yet

1 participant