Skip to content

Conversation

@markpolyak
Copy link
Owner

No description provided.

markpolyak and others added 7 commits January 16, 2023 10:24
Add photo
Add information about people
Add information about people
Add information about people
Copy link
Collaborator

@KleinerKaktus455 KleinerKaktus455 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Заметила, что ссылки на фотографии не везде корректны (например: фото 39, 3). Необходимо пересмотреть все ссылки и исправить их на корректные
  2. Насколько я знаю, Марк Дмитриевич отправил Вам перечень сайтов, с которых нужно взять фотографии. Почему-то часть фотографий взята с сайтов, которые отсутствуют в перечне. Удалите такие фотографии и замените их на изображения с указанных сайтов. Если найденное изображение не соответствует требованиям из документа, то просто проигнорируйте его, не пытайтесь найти фото этого музыканта в другом источнике
  3. Все студенты собирали 100 фотографий, у Вас всего 92. Добейте, пожалуйста, до нужного количества

@KleinerKaktus455 KleinerKaktus455 self-requested a review February 4, 2023 18:48
@Tony-StPet
Copy link

https://github.com/Tony-StPet/orchestra_musician
Не получается создать папку в древе.

@KleinerKaktus455
Copy link
Collaborator

Что Вы подразумеваете под "папкой в древе?" Речь про ветку?

@KleinerKaktus455
Copy link
Collaborator

@Tony-StPet
Уже могу сказать, что задание однозначно не будет принято, т.к. Вы проигнорировали практически все требования к изображениям.

  1. Необходимо брать фотографии из соц сетей, а не с сайтов симфонических оркестров
  2. Вы проигнорировали требование насчет фона изображений и профессиональных фотосессий. Подходят, например, изображения 14 и 20 из Вашего датасета
  3. Вы не обрезали фотографии
    Перечитайте еще раз задание, описанное здесь и выполните его качественно.

@Tony-StPet
Copy link

Переделал, согласно заданию, ветвь orchestra_musician
/symphony_personal2 — копия
Проверьте, пожалуйста, если все норм, то в процессе доделаю остальные фото и ссылки

@KleinerKaktus455
Copy link
Collaborator

Посмотрела изображения из readme (36 картинок). То, что нужно! Теперь буду ждать оформления файла с описанием

@Tony-StPet
Copy link

50 изображений и файл описание. Готово. Если будет время, могу увеличить количество до 100. все зависит от МД

@KleinerKaktus455
Copy link
Collaborator

На будущее: очень рекомендую Вам делать информативные сообщения к коммитам ("add" не подойдет, сами потом запутаетесь). Это хорошая привычка, не игнорируйте этот момент. Это совет, не замечание.

Насчет работы: все ок, проверила описание. Насчет кол-ва фото последнее слово будет за Марком Дмитриевичем, я проверяю только качество выполнения задания

@markpolyak, ситуация следующая:
С качеством выполнения задания все ок. Насчет недостатков работы:

  1. Не соблюдены требования насчет создания ветки и создания PR. У исполнителя по каким-то причинам не получилось создать ветку, судя по нашей переписке здесь. Вся работа выполнена в личном репозитории.
  2. На данный момент собрано 50 фотографий. Не знаю, на какое количество вы договаривались

Итого: не соблюдены требования по работе в GitHub, к фотографиям претензий нет.

@markpolyak
Copy link
Owner Author

@KleinerKaktus455 работа сдана, пул-реквест можно мерджить

@KleinerKaktus455
Copy link
Collaborator

@markpolyak , это не его пул, мерджить нечего, т.к. отдельная ветка не создана( Потом самостоятельно залью изменения

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.

Подготовка выборки лиц: симфонические музыканты

5 participants