search

DCGM / pero_ocr_web

BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

layout displayed in leaflet does not match the page #34

Closed zabak closed 4 years ago

zabak commented 4 years ago

image https://pero-ocr.fit.vutbr.cz/layout_analysis/show_results/38a7cfa6-5da6-4d6c-a840-b2d5a6563fe3

michal-hradis commented 4 years ago

We have updated layout editor code. Could you check if this problem persists?

zabak commented 4 years ago

well... I see no change in chrome so I tested Firefox and Safari and the result is that the preview page is still too wide and in Leaflet it is turned 90 degrees (it probably ignores the image orientation info in the jpg file).

image

zabak commented 4 years ago

The first page is incorrectly displayed too wide. Both pages are turned 90°in Leaflet

michal-hradis commented 4 years ago

Could you share the document with me? (add me as a collaborator) The preview look correct - previews are the same height and the whole page is shown.

zabak commented 4 years ago

Shared already. Currently the images are not rotated, but still the preview image is too wide. They are both from the same exercise book - if you count number of lines there is some difference but not that much. And the blue line outlines are proportioned correctly. It is just the image itself which is deformed.

zabak commented 4 years ago

All the images were taken in portrait orientation and not cropped.

michal-hradis commented 4 years ago

OK. This issue is fixed for newly uploaded images. Leaflet ignores jpeg EXIF orientation flags. Current solution re-compresses all jpeg images during upload and removes the orientation flag.

This does not FIX already uploaded images. 722c216fafddfe0cd9e78eb1f8b58b8e6bdf25ab

michal-hradis commented 4 years ago

We fixed already uploaded images. The live app should show all images correctly. If you see some rotated pages, update your browser :).