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

Updated readme #314

Merged
merged 38 commits into from
Dec 22, 2022
Merged

Updated readme #314

merged 38 commits into from
Dec 22, 2022

Conversation

valentina-kustikova
Copy link
Contributor

Обновлены основной редми-файл в соответствии с текущим состоянием на Wiki и файлы, ссылки на которые идут с Wiki.

@valentina-kustikova
Copy link
Contributor Author

valentina-kustikova commented Dec 11, 2022

@FenixFly, проверь, пожалуйста, внимательно, что все нормально в файле с перечнем моделей.

  1. У тебя где-то таблицы были без шапок и еще какие мелочи.
  2. Модель deblurgan-v2 указана в двух таблицах, это нормально?
  3. Еще, мне кажется, что ты удалил некоторые модели, которые у нас были в предыдущих версиях провалидированы. Это неправильно, надо было их оставить и просто поставить прочерк, что их сейчас нет в OMZ. Иначе получается, что в табличках с массовыми экспериментами они где-то появлялись, мы их валидировали, но что это за модели, мы не знаем. Просьба, если такие действительно есть, их восстановить.

На будущее просьба - запрашивать ревью своих PR перед мержем. Спасибо!

@valentina-kustikova
Copy link
Contributor Author

@a-sidorova, посмотрите, пожалуйста, wiki проекта, основные вещи (то, что удалено) я перенесла туда.

@FenixFly
Copy link
Contributor

FenixFly commented Dec 11, 2022

@FenixFly, проверь, пожалуйста, внимательно, что все нормально в файле с перечнем моделей.

1. У тебя где-то таблицы были без шапок и еще какие мелочи.

2. Модель deblurgan-v2 указана в двух таблицах, это нормально?

3. Еще, мне кажется, что ты удалил некоторые модели, которые у нас были в предыдущих версиях провалидированы. Это неправильно, надо было их оставить и просто поставить прочерк, что их сейчас нет в OMZ. Иначе получается, что в табличках с массовыми экспериментами они где-то появлялись, мы их валидировали, но что это за модели, мы не знаем. Просьба, если такие действительно есть, их восстановить.

На будущее просьба - запрашивать ревью своих PR перед мержем. Спасибо!

Да, я удалил несколько моделей, которые мы не валидировали и не бенчмаркали, которые сейчас отсутвуют в OpenVINO, например:
person-reidentification-retail-0031|-|нет доступа к набору данных Market-1501|-|-|
person-reidentification-retail-0076|-|нет доступа к набору данных Market-1501|-|-|
person-reidentification-retail-0079|-|нет доступа к набору данных Market-1501|-|-|
(Отсутствуют с 2019-R2)

face-recognition-mobilefacenet-arcface|-|не известен тренировочный набор данных|-|-|
face-recognition-resnet100-arcface|-|не известен тренировочный набор данных|-|-|
face-recognition-resnet34-arcface|-|не известен тренировочный набор данных|-|-|
face-recognition-resnet50-arcface|-|не известен тренировочный набор данных|-|-|
(Отсутствуют с 2021.4)

Их вернуть?

@valentina-kustikova
Copy link
Contributor Author

@FenixFly, проверь, пожалуйста, внимательно, что все нормально в файле с перечнем моделей.

1. У тебя где-то таблицы были без шапок и еще какие мелочи.

2. Модель deblurgan-v2 указана в двух таблицах, это нормально?

3. Еще, мне кажется, что ты удалил некоторые модели, которые у нас были в предыдущих версиях провалидированы. Это неправильно, надо было их оставить и просто поставить прочерк, что их сейчас нет в OMZ. Иначе получается, что в табличках с массовыми экспериментами они где-то появлялись, мы их валидировали, но что это за модели, мы не знаем. Просьба, если такие действительно есть, их восстановить.

На будущее просьба - запрашивать ревью своих PR перед мержем. Спасибо!

Да, я удалил несколько моделей, которые мы не валидировали и не бенчмаркали, которые сейчас отсутвуют в OpenVINO, например: person-reidentification-retail-0031|-|нет доступа к набору данных Market-1501|-|-| person-reidentification-retail-0076|-|нет доступа к набору данных Market-1501|-|-| person-reidentification-retail-0079|-|нет доступа к набору данных Market-1501|-|-| (Отсутствуют с 2019-R2)

face-recognition-mobilefacenet-arcface|-|не известен тренировочный набор данных|-|-| face-recognition-resnet100-arcface|-|не известен тренировочный набор данных|-|-| face-recognition-resnet34-arcface|-|не известен тренировочный набор данных|-|-| face-recognition-resnet50-arcface|-|не известен тренировочный набор данных|-|-| (Отсутствуют с 2021.4)

Их вернуть?

@FenixFly, нет. Я же написала только про те модели, которые были нами провалидированы, а из OMZ их удалили. Если тавые были, то вот их надо вернуть, потому что они у нас фигурируют в валидационных таблицах.

@valentina-kustikova
Copy link
Contributor Author

@FenixFly, @a-sidorova, внесла изменения из PR, можно смотреть.

@a-sidorova
Copy link
Contributor

Насчет Wiki я, пожалуй, оставлю тут комментарии, ибо я не совсем понимаю, как ревьювить wiki-страницы. Там вроде можно сразу лишь редактировать.

  • BuildForLinux. Вроде как intel caffe устанавливается иначе. Я даже помню, что там было миллион проблем из-за того, что это устаревший продукт. В DockerFile более подробно указано, как устанавливается фреймворк (это кстати также взято из DockerFile репозитория caffe). Я это к тому, что строчка conda install -c intel caffe, что взята из репозитория самоого caffe, возможно не совсем достаточна. Более того, я не совсем уверена, что intel-caffe поддерживает python3.8. В общем, я бы предложила создать issue, чтобы провалидировать intel-caffe
  • QuickDeploymentGuide. Насчет 7 пункта. У нас есть маунтирование директории с моделями и наборами данных в докер контейнер при его запуске. Ничего руками копировать не нужно. Так что выглядит так, что 7 пункт - лишний. Поправьте, пожалуйста.
  • QuickDeploymentGuide. 3 пункт примера. Это уже устаревший пример DockerFile. На текущий момент ссылки на OpenVINO уже нет, так что ARG DOWNLOAD_LINK можно удалить
  • QuickStartupGuide. Пример конфигурационного файла для бенчмаркинга некорректен. На данный момент нужно отдельно указывать путь до модели и ее весов. Также путь до модели в accuracy checker, логично предположить, должен быть такой же, как в конфигурационном файле для бенчмаркинга, поскольку мы до этого примаунтили директорию с моделями /mnt/models
  • Результаты. В табличках под столбцом tag ссылки на скачивание архивов. Под именем tag я бы наверно ожидала увидеть имя тэга, а под именем archive ссылку на скачивание сборки. Что думаете?

@valentina-kustikova
Copy link
Contributor Author

Насчет Wiki я, пожалуй, оставлю тут комментарии, ибо я не совсем понимаю, как ревьювить wiki-страницы. Там вроде можно сразу лишь редактировать.

  • BuildForLinux. Вроде как intel caffe устанавливается иначе. Я даже помню, что там было миллион проблем из-за того, что это устаревший продукт. В DockerFile более подробно указано, как устанавливается фреймворк (это кстати также взято из DockerFile репозитория caffe). Я это к тому, что строчка conda install -c intel caffe, что взята из репозитория самоого caffe, возможно не совсем достаточна. Более того, я не совсем уверена, что intel-caffe поддерживает python3.8. В общем, я бы предложила создать issue, чтобы провалидировать intel-caffe

Создала issue#320 по тестированию мануалки, туда же вписала задачу по тестированию сборки.

  • QuickDeploymentGuide. Насчет 7 пункта. У нас есть маунтирование директории с моделями и наборами данных в докер контейнер при его запуске. Ничего руками копировать не нужно. Так что выглядит так, что 7 пункт - лишний. Поправьте, пожалуйста.

Спасибо, исправила.

  • QuickDeploymentGuide. 3 пункт примера. Это уже устаревший пример DockerFile. На текущий момент ссылки на OpenVINO уже нет, так что ARG DOWNLOAD_LINK можно удалить

Удалила.

  • QuickStartupGuide. Пример конфигурационного файла для бенчмаркинга некорректен. На данный момент нужно отдельно указывать путь до модели и ее весов. Также путь до модели в accuracy checker, логично предположить, должен быть такой же, как в конфигурационном файле для бенчмаркинга, поскольку мы до этого примаунтили директорию с моделями /mnt/models

Исправила.

  • Результаты. В табличках под столбцом tag ссылки на скачивание архивов. Под именем tag я бы наверно ожидала увидеть имя тэга, а под именем archive ссылку на скачивание сборки. Что думаете?

Заменила слова в шапках таблиц с результатами измерения производительности и качества.

@a-sidorova, проверьте, пожалуйста, все ли так. Я еще перенесла вашу мануалку на Wiki, пока не переводила.

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

Successfully merging this pull request may close these issues.

3 participants