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

feature: deskew before dewarping #26

Open
zvezdochiot opened this issue Feb 11, 2024 · 2 comments
Open

feature: deskew before dewarping #26

zvezdochiot opened this issue Feb 11, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@zvezdochiot
Copy link
Member

Hi @plzombie , @noobie-iv .

Deskew before dewarping.

Имеется определённый смысл в применении и регулировке deskew перед перспективным искажением или искажениям по кривым. Это не только снизит "эффекты" алгоритмов автоопределения направляющих, но и в случае нужды позволит вернуть сортировку по углу поворота. Плюс к этому это позволит привести всё хозяйство к единной и более понятной форме отображения.

Конечно же, не хочется "ломать", то что и так работает, но хотя бы "посмотреть" в эту сторону не мешает.

@zvezdochiot zvezdochiot self-assigned this Feb 11, 2024
@zvezdochiot zvezdochiot added enhancement New feature or request help wanted Extra attention is needed labels Feb 11, 2024
@noobie-iv
Copy link
Member

У меня вообще, глядя на скорость, с которой несчастный сплайн по 4 точкам вписывается в кривую, появляется желание переделать все "как надо", убрав рекурсивный алгоритм вместе с этим нашим недобитым багом. Не может быть, чтобы у такой задачи не было "почти мгновнного" решения. Возможно, оно вообще гуглится. Тормоза с деварпом посреди реактивного STEX реально бесят.

@zvezdochiot
Copy link
Member Author

zvezdochiot commented Feb 11, 2024

@noobie-iv say:

желание переделать все "как надо"

Только ты пили это в STD, ибо более перспективно. А уж сюда как-нибудь сможем перетащить. Но ежели наладить STD - то это точно будет огонь!

PS: А ежели пользовать сплайны Акимы вместо B-сплайнов, то это вообще будет огнище! Но для сплайнов Акимы точно будет более чем требуем предварительный deskew.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants