-
Notifications
You must be signed in to change notification settings - Fork 37
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
[Demo] Added linux script #307
[Demo] Added linux script #307
Conversation
4deaaa9
to
191a371
Compare
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.
@a-sidorova, если 2 соображения для обсуждения:
- Насколько я понимаю, мы можем запускать бенчмарк без докера. Поэтому я бы сделала 2 режима запуска демо-приложения, которые запускались с / без докера.
- Система поддерживает вывод с использованием нескольких фреймворков, не только OpenVINO, поэтому хотелось бы конечно демонстрацию для других фреймворков. Возможно, это также отдельный параметр скрипта или вовсе разные демо (надо подумать, как лучше).
Думаю, что наверно стоит иметь для каждого фреймворка свою директорию, внутри которой будет соответствующие скрипты под каждую ОС. Предлагаю сделать для этого issue и в следующем семестре это реализовать, поскольку времени остается мало, и, честно говоря, большую часть времени ушло на поднятие окружения, создание конфигурационных файлов, дебаг и фикс всяких баг в процессе сбора результатов экспериментов для 2022.1. |
@a-sidorova, я не возражаю, написала, поскольку это надо сделать. Создайте, пожалуйста, issue, чтобы не забыть. Единственное - предлагаю сейчас уже предусмотреть структуру директорий и название демо поменять. |
@FenixFly, надо отревьюить изменения. |
@valentina-kustikova создала новую директорию. Думаю, что под каждую OS можно реализовать свой скрипт, где с помощью параметра можно регулировать запуск: host, docker. Также создала тикет на реализацию остальных демо-приложений |
@a-sidorova, да, это правильно. |
demo/OpenVINO_DLDT/README.md
Outdated
|
||
- `-l` - логин для подключения к текущей машине. | ||
- `-p` - пароль для подключения к текущей машине. | ||
- `-d` - адрес до GitHub-репозитория, который содержит в себе набора данных для бенчмаркинга. |
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.
...набор данных...
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.
Поправила, спасибо
demo/OpenVINO_DLDT/linux.sh
Outdated
|
||
|
||
echo "[ INFO ] System cleaning has been started" | ||
sudo apt-get remove -y python3-tk docker.io |
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.
Слишком жестко удалять докер из системы, если он до этого стоял. Надо сделать какую-то булевую переменную, которая будет сигнализировать о том, стоял докер до этого или нет. Если не стоял, то удалить.
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.
Отличное замечание! Добавила, спасибо!
demo/OpenVINO_DLDT/README.md
Outdated
|
||
- `-l` - логин для подключения к текущей машине. | ||
- `-p` - пароль для подключения к текущей машине. | ||
- `-d` - адрес до GitHub-репозитория, который содержит в себе набора данных для бенчмаркинга. |
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.
Думаю стоит добавить описание того, что должно быть в репозитории, поскольку наш репозиторий с данными закрытый: "Для демонстрационного приложения в GitHub-репозитории набора данных должна быть как минимум директория с названием ImageNET, в которой будет хотя бы X(?) изображений"
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.
Верно. Добавила комментарий, спасибо!
demo/OpenVINO_DLDT/linux.sh
Outdated
PYTHON="${venv_path}/bin/python3" | ||
$PYTHON -m pip install -r $root_folder/requirements.txt | ||
echo "[ INFO ] Python environment ${python3} has been created" | ||
sudo apt-get install -y python3-tk docker.io |
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.
добавьте пожалуйста к установке еще containerd
, его требует docker.io, но не при уставновке, а при запуске
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.
Хорошо, добавила. Спасибо!
demo/OpenVINO_DLDT/linux.sh
Outdated
echo "[ INFO ] Cloning of OMZ repository" | ||
omz_client="${client_folder}/open_model_zoo" | ||
[ -d $omz_client ] && rm -rf $omz_client | ||
git clone https://github.com/openvinotoolkit/open_model_zoo.git --recursive --branch $openvino_version --single-branch |
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.
Для ускорения скачивания надо во все команды got clone добавить параметр --depth 1
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.
Отлично, спасибо!
demo/OpenVINO_DLDT/linux.sh
Outdated
<Computer> | ||
<IP>${ip_address}</IP> | ||
<Login>${login}</Login> | ||
<Password>${password}</Password> |
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.
@a-sidorova, поехало форматирование в xml.
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.
Да, как обычно конфликт между пробелами и табуляцией, поправила. Спасибо
demo/OpenVINO_DLDT/linux.sh
Outdated
echo \ | ||
"<?xml version=\"1.0\" encoding=\"utf-8\"?> | ||
<Tests> | ||
<Test> |
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.
Здесь тоже как-то странное едет форматирование. Видимо, стоит табуляция вместо пробелов.
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.
Поправила
demo/OpenVINO_DLDT/linux.sh
Outdated
echo "[ INFO ] Downloading of dataset 'cifar-10-python'" | ||
wget https://www.cs.toronto.edu/~kriz/cifar-10-python.tar.gz | ||
mkdir dataset && tar xvf cifar-10-python.tar.gz -C dataset | ||
datasets_dir="${client_folder}/datasets" |
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.
В строке 72 используется директория dataset, а в строке 73 директория datasets
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.
Поправила, спасибо!
@a-sidorova, надо обновиться до состояния master-ветки, чтобы исправить ошибки в стиле кодирования. |
843b712
to
58764b1
Compare
@valentina-kustikova отребейзилась, взгляните еще раз, пожалуйста |
Blockers