Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

debug multiple classes showing up on binary synthetic data - facilitates correct mask visualizations representative of the number of classes #326

Closed
yalaudah opened this issue May 28, 2020 · 1 comment · Fixed by #345
Closed

debug multiple classes showing up on binary synthetic data - facilitates correct mask visualizations representative of the number of classes #326

yalaudah opened this issue May 28, 2020 · 1 comment · Fixed by #345
Assignees
@yalaudah
Labels
Prior: ShowStopper Blocks a release Status: In-progress Active on-going work in the sprint Type: Bug Something isn't working Type: Correctness anything to do with repo being technically correct
Milestone
V0.1.3 [May release]

Comments

@yalaudah
Copy link
Contributor

yalaudah commented May 28, 2020 •
edited
Loading

Investigate why the model predictions in the binary dataset result in images with multiple classes like this image:
image
image

This might be due to the logits being passed on to mask_to_disk() before taking the max()
@yalaudah yalaudah changed the title Debug binary dataset outputs Debug binary dataset images and model outputs May 28, 2020
@yalaudah yalaudah changed the title Debug binary dataset images and model outputs debug multiple classes showing up on binary synthetic data - facilitates correct mask visualizations representative of the number of classes May 28, 2020
@yalaudah yalaudah added this to the V0.1.3 [BYOD] milestone May 28, 2020
@yalaudah yalaudah added Prior: ShowStopper Blocks a release Type: Bug Something isn't working Type: Correctness anything to do with repo being technically correct labels May 28, 2020
@yalaudah yalaudah self-assigned this Jun 2, 2020
@yalaudah yalaudah added the Status: In-progress Active on-going work in the sprint label Jun 2, 2020
@yalaudah yalaudah mentioned this issue Jun 4, 2020
Merged
@yalaudah yalaudah linked a pull request Jun 4, 2020 that will close this issue
Minor fix to address #326 #345
Merged
@yalaudah
Copy link
Contributor Author

yalaudah commented Jun 4, 2020 •
edited
Loading

I removed the other issue (binary image becomes non binary after augmentation) since it is unrelated. It is now referenced in issue: #344

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@yalaudah yalaudah

Labels
Prior: ShowStopper Blocks a release Status: In-progress Active on-going work in the sprint Type: Bug Something isn't working Type: Correctness anything to do with repo being technically correct
Projects
None yet
Milestone
V0.1.3 [May release]
Development

Successfully merging a pull request may close this issue.

Minor fix to address #326 microsoft/seismic-deeplearning
2 participants
@yalaudah @maxkazmsft