-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[NNAdapter] Add 35 unit tests for model validation from PaddleClas, PaddleDetection, PaddleSeg, PaddleOCR etc. #8401
[NNAdapter] Add 35 unit tests for model validation from PaddleClas, PaddleDetection, PaddleSeg, PaddleOCR etc. #8401
Conversation
Thanks for your contribution! |
43c4488
to
18e50f9
Compare
18e50f9
to
1129347
Compare
CHECK_EQ(static_cast<size_t>(file_size), | ||
static_cast<size_t>(image_size) * sizeof(T) / sizeof(char)); | ||
fin.read(reinterpret_cast<char*>(data), file_size); | ||
fin.read(reinterpret_cast<char*>(data), image_size * sizeof(T)); |
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.
为什么把这个check去掉?
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.
因为新加的检测模型的输入有各种shape大小,现在数据集的raw数据就608x608。后续准备改写成直接读取图片,每个模型单测用例自行做resize和预处理。
而当前检测模型主要是跑通,精度暂时不验证。所以就按上述原因去掉了check。
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.
LGTM
PaddleClas
PaddleDetection
PaddleSeg
PaddleOCR
PaddleRec
PaddleVideo