search

ImageProcessing-ElectronicPublications / scantailor-deviant

ScanTailor Deviant - a fork based on Universal and Experimental families of ST
https://github.com/trufanov-nok/scantailor-universal
GNU General Public License v3.0
6 stars 0 forks source link

dewarping: top curve: doesn't work == line, 2 points. #2

Open zvezdochiot opened 10 months ago

zvezdochiot commented 10 months ago

Hi @noobie-iv .

Уже много раз на текущем состоянии git наблюдаю прямую из двух точек по верхней грани при выборе "Кривых" в "Устранении искажений". Нижняя грань, как и положено, из 5-ти точек. В верхнюю грань точки можно добавить, но как то это "слегка" не очень. "Совпадение? Не думаю..." STD STEX
screen-3 screen02

B_199.origin.tif.gz [2635403] (SHA1: 2c801c5d5df0bed28aec2651b1331385c8aebe57)

PS: Да и вообще, хотелось бы иметь выбор кол-ва точек по граням, скажем от 2 до 16.

noobie-iv commented 10 months ago

До начала работы над переносом деварпа я делал тестовый проект, где только деварп есть, а остальное все вырезано, просто чтобы оценить объем. В нем было больше мегабайта исходников, чтобы только собираться. И это без проверки, правильно ли он работает; возможно, я что-то зевнул, и часть исходников потерял.

То есть, чтобы перенести деварп из STEX абсолютно точно, надо заменить около трети объема STU (или продублировать этот объем внутрь папки с деварпом). Это явный перебор, делать из STU обратно STEX точно не годится.

Поэтому при конфликтах переноса я принудительно оставлял все, что в принципе собиралось и работало в STU, и переносил только код, которого в STU совсем не было. Однозначно, в каких-то функциях есть расхождения в реализации, которые потерялись при переносе, и будут давать расхождения. Например, я поправил ляп в многопоточном коде построения гистограмм в STU, из-за которого Watershed - фильтр не работал, и верхняя линия постоянно привязывалась к верху страницы вместо текста.