Update validation_results_openvino_public_models.md #560
Conversation
|
@SergeyBoRss, пулл-реквест находится в состоянии draft, поэтому я мержить его не могу. В какой момент вы планируете замержить изменения? Я думала отдельно залить табличку и отдельно адаптеры. |
Я планировал ещё для YOLO модели сделать адаптер и соответственно добавить в таблицу, но возникли сложности, перевёл из драфта |
src/inference/io_adapter.py
Outdated
| min(int(class_id * 12.5), 255), | ||
| min(class_id * 7, 255), | ||
| min(class_id * 5, 255), | ||
| ) |
There was a problem hiding this comment.
А зачем нам генерация цвета в обработчике результата детектирования?
There was a problem hiding this comment.
Делал для цвета, зависящего от класса (взял как пример адаптер DetectionIO). Сейчас заменил на 1 цвет
There was a problem hiding this comment.
Нам на выходе вообще цвета не нужны, потому что это касается отображения, а не результатов детектирования. При необходимости цвет может быть сгенерирован тем, кто вызывает детектор.
src/inference/io_adapter.py
Outdated
| min(class_id * 7, 255), | ||
| min(class_id * 5, 255), | ||
| ) | ||
| boxes_dict[image_number].append(((xmin, ymin), (xmax, ymax), score, class_id, color)) |
There was a problem hiding this comment.
Аналогичный вопрос. Вроде бы нигде не используем.
src/inference/io_adapter.py
Outdated
| log.info('Result image was saved to {0}'.format(out_img)) | ||
| count += 1 | ||
|
|
||
| def _non_max_suppression(self, boxes, iou_threshold=0.3): |
There was a problem hiding this comment.
А нельзя использовать стандартную функцию cv2.dnn.NMSBoxes для подавления не-максимумов? Внешне ваша функция делает то же самое, что и указанная.
There was a problem hiding this comment.
Да, действительно можно использовать её, немного переписал код для этого
| @@ -0,0 +1,48 @@ | |||
| { | |||
There was a problem hiding this comment.
Этот файл не надо выкладывать в репозиторий.
| @@ -0,0 +1,48 @@ | |||
| { | |||
There was a problem hiding this comment.
И этот файл тоже не нужен в репозитории.
|
@SergeyBoRss, подтяните, пожалуйста, основную ветку. |
Готово, обновил, также добавил адаптер для YoloV3-tiny |
|
@ismukhin, посмотрите, пожалуйста. |
|
@SergeyBoRss, а вы пробовали запустить первые yolo-модели, для которых уже имеются адаптеры? В целом, чтобы работа выглядела последовательной, имеет смысл запустить, проверить работоспособность и при необходимости что-то исправить. |
SergeyBoRss
dismissed stale reviews from ismukhin and valentina-kustikova
via
035fde8
Да, я запускал часть моделей из openvino, почти все текущие адаптеры работают, но мне неясно зачем необходим адаптер YoloV3TFIO, который к тому же и не работает, ведь для модели yolo-v3-tf уже работает адаптер yolo_v3 |
Интересует, какие конкретно адаптеры для yolo на данный момент не работают? Касательно YoloV3TFIO, то это либо адаптер, который предназначен для обработки выхода старой модели yolo-v3, либо он имеет отношение к другим фреймворкам. |
No description provided.