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

Resample mask error #19

Closed
vankhoa21991 opened this issue Jul 19, 2021 · 2 comments
Closed

Resample mask error #19

vankhoa21991 opened this issue Jul 19, 2021 · 2 comments

Comments

@vankhoa21991
Copy link

Hello, I reveived this warning and error in the preprocessing stage. This is due to the resample process that make some instances disappear?

2021-07-19 15:06:43.138 | WARNING | nndet.planning.properties.instance:instance_class_and_region_sizes:213 - Instance lost. Found {1: 0, 2: 0, 3: 0, 4: 0} in properties but [1 2 3] in seg of 1_2_840_113654_2_55_446543879.

Thanks

@vankhoa21991 vankhoa21991 changed the title [Question] Resample mask error Jul 19, 2021
@vankhoa21991
Copy link
Author

When I read the code, it seems like at this stage the only thing was done is to read image and save back to npz, so what cause is the reason of instance missing??

@mibaumgartner
Copy link
Collaborator

Hi @vankhoa21991 ,

can you double check that specific case for any anomalies (e.g. wrong segmentation mask, wrong label json, data corruption due to an error during a copy process)? Since the warning is raised in the analysis phase no resampling was performed until that point. The case was only cropped to the non-zero regions so all instances should be present during analysis. (Sometimes if the data is corrupted, large regions of the scan were zero and thus disappeared in the cropping stage).

You are right that small objects can vanish during the resampling phase though (and that also happens sometimes). The message looks different then and can be found here

logger.warning(f"Inconsistent instances {case_npz} from "
f"properties {props_instances} from seg {seg_instances}. "
f"Very small instances can get lost in resampling "
f"but larger instances should not disappear!")
.

Best,
Michael

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

No branches or pull requests

2 participants