-
Notifications
You must be signed in to change notification settings - Fork 709
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
Minor fixes #1182
Minor fixes #1182
Conversation
…/anomalib into efficientad_quantile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @alexriedel1, some minor comments only
I have just one question. Does that extra dimension really cause issues for pixel metrics? I believe that I haven't encountered that, but it is possible that this is due to nature of ROC where entire tensor gets flattened anyway. |
I had some cases where the additional dimension in the heat maps caused an error. |
done! |
ok now its done i guess :) |
Description
It can happen for some models, that the output anomaly map can have an extra dimension [8,1,256,256] whereas the ground truth mask has not. This causes an error. To fix this, the squeeze function is added before calculating pixel metrix.
This PR uses an ImportWarning instead of an ImportError for missing OpenVINO install. Otherwise without OpenVINO installed anomalib will crash.
Also this PR makes a change on how to read augmentation config from the config file to make image size HPO sweeps more accesible.
Also the PR adresses #1177 and some other issues with the tensor shape for torch.quantile being too large in the efficientad torch model during training
Changes
Checklist