Skip to content

Deep label fusion - #124

Open
EmileSaillard wants to merge 10 commits into
mainfrom
Deep-Label-Fusion
Open

Deep label fusion#124
EmileSaillard wants to merge 10 commits into
mainfrom
Deep-Label-Fusion

Conversation

@EmileSaillard

Copy link
Copy Markdown
Contributor

No description provided.

@EmileSaillard
EmileSaillard requested a review from maylis-j March 28, 2023 07:46

@maylis-j maylis-j 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.

I mostly corrected some typos, and commented when i thought changes would make the whole review more clear.

Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md
EmileSaillard and others added 6 commits March 28, 2023 11:19
Co-authored-by: maylis-j <73818320+maylis-j@users.noreply.github.com>
Co-authored-by: maylis-j <73818320+maylis-j@users.noreply.github.com>
Co-authored-by: maylis-j <73818320+maylis-j@users.noreply.github.com>
Co-authored-by: maylis-j <73818320+maylis-j@users.noreply.github.com>
Co-authored-by: maylis-j <73818320+maylis-j@users.noreply.github.com>

@nathanpainchaud nathanpainchaud left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great review! Just a few fixes for the references and a word.

Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
Comment thread collections/_posts/2023-03-14-Deep-Label-Fusion.md Outdated
EmileSaillard and others added 2 commits April 11, 2023 17:24
Co-authored-by: Nathan Painchaud <23144457+nathanpainchaud@users.noreply.github.com>
@nathanpainchaud

Copy link
Copy Markdown
Member

Since the version reviewed is only available through MedIA and not as an arXiv paper, I think we should not merge this branch right now (at least if we follow the same policy we've had up until now).

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.

3 participants