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

Головы у спикеров обрезаются #224

Open
vadlit opened this issue Feb 28, 2019 · 7 comments
Open

Головы у спикеров обрезаются #224

vadlit opened this issue Feb 28, 2019 · 7 comments

Comments

@vadlit
Copy link
Contributor

vadlit commented Feb 28, 2019

Головы у спикеров обрезаются. Android, Asus Zenfone 5z
Скрин https://drive.google.com/file/d/1DnCQwTn8vkfQnph-wcc5ec8v7XpPj7pk/view?usp=drivesdk

На виртуалке такого не замечал

@kulakovt
Copy link
Member

Не могу посмотреть скрин, не хватает прав

@vadlit
Copy link
Contributor Author

vadlit commented Mar 1, 2019

А теперь?

@kulakovt
Copy link
Member

kulakovt commented Mar 1, 2019

@vadlit Теперь видно, спасибо.

@pfedotovsky
Copy link
Contributor

pfedotovsky commented Mar 3, 2019

@vadlit @AnatolyKulakov
К сожалению, вариантов тут немного для отображения фотографий:

  1. Показывать полную фотографию сохраняя масштаб, но возможно слева и справа будут пустые полосы
  2. Не сохранять масштаб, тогда фотографии будут вытянуты
  3. Сохранять масштаб, но обрезать фотографию по краям

Варианты 1 & 2 визуально выглядят плохо, поэтому сейчас используется 3.

Самый простой вариант - изменить фотографию. Вариант посложнее - использовать AI, чтобы при обрезании голова оставалась (например сервисы Azure, надо смотреть)

@kulakovt
Copy link
Member

kulakovt commented Mar 3, 2019

Проблема понятна. Я копал именно в сторону Ажуроского распознавания лиц. Ибо это также решает несколько других проблем. Например создание small фотографий, на которых видно только лицо спикера. Но у меня не хватило времени довести это до конечного результата. Думаю нужно создать отдельную issue на тулзу по определению и обрезанию лиц.

@kulakovt
Copy link
Member

kulakovt commented Mar 6, 2019

Это гениально:
https://demo.cloudinary.com/auto-gravity

cloudinary.Api.UrlImgUp.Transform(
	new Transformation().Width("192").Crop("thumb").Gravity("auto").Height("247")
).Format("jpg").BuildImageTag("woman");

@kulakovt
Copy link
Member

Я перепроверил проблему и дело вовсе не в масштабе. Дело в заголовке "Speaker". Он перекрывает фотографию, хотя должен отодвигать. Фотография спикера должна начинаться там где заканчивается заголовок. А сейчас она начинается там где начинается и заголовок и подлезает под него.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants