-
Notifications
You must be signed in to change notification settings - Fork 73
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
found a small formal bug #2
Comments
Hi @mg64ve ! Yes thank you for finding out this bug, I have corrected it. It had indeed no impact since batch_size was already defined earlier. |
You are welcome @vincent-leguen.
and at line 120 you get breaks:
How do you use this values? I don't see any use of them. |
Hi @mg64ve , I added these additional breakpoints when I wanted to add new external metrics for the paper (here the Hausdorff distance). But for simplicity of the published code, I removed the computation of external metrics so the breakpoint piece of code is not useful here. |
Ok thanks @vincent-leguen |
Hi @vincent-leguen,
nice work, congrats.
I am still studying the way you calculate DILATE loss.
However I think there is a small formal bug in main.py at row 78:
should be:
it is a small typo, but I do't think it impacts the normal flow of the code.
Cheers
The text was updated successfully, but these errors were encountered: