-
Notifications
You must be signed in to change notification settings - Fork 9
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 #47: binarizeEdgePlus #48
feature #47: binarizeEdgePlus #48
Conversation
Hi @vigri , @lightsilverberryfox . The usefulness of the new thresholds has been confirmed (worst part of the scanned material):
See threshold |
[Feature request] thresholdAdjustment in other threshold. #59 |
Mod epdbta of release 1.0.18: https://github.com/ImageProcessing-ElectronicPublications/scantailor-advanced/releases/tag/v1.0.18 |
Hi! |
@alfish2000 say:
I don't hope so. Therefore, I release mods for existing releases. ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, the code seems to be fine, I don't understand how the EdgePlus algorithm works though... But if it works it should be okay. Let's see if @vigri has something to say.
I'm confused by your "I don't hope so." |
Hi @mara004 . I don't have enough experience with Qt and XML. If there are bugs, it will be difficult for me to eliminate them. If this PR enters the mainstream, I will be happy. Doubts are caused purely by lack of skill (especially in XML). PS: During testing, I did not notice any failures. |
…efilters EdgePlus and BlurDiv.
Hi @mara004 , @lightsilverberryfox , @vigri . Github Action: build-linux-qt5:
http://azure.archive.ubuntu.com/ubuntu/pool/main/libi/libinput/:
Thu, 13 Jul 2023 15:49:44 +0300 Resolved: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, @zvezdochiot
After some testing and reviewing, I believe the PR is ready to be merged, I didn't encounter any issues and this could be a very useful feature, I would normally wait for @vigri to offer their review, however he hasn't been here for three months already, so, I don't know what to do.
If you believe it's ready to be merged, I'll merge it. I can tell you put a lot of effort in this PR and wanted to say thank you, you have also offered your input on other issues and bugs, so if vigri sees this, he may want to consider sending you an invitation to our organization so you have write access (I can't invite you myself, I don't have owner permissions).
Is issue #59 also solved by this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall I see no problems. Please see my comments. Especially the one with the qt-version.
apologies very much. Thank you very much @zvezdochiot. Regarding this PR: Pleasse see my "review". |
@lightsilverberryfox say:
This PR closes the issues:
PS: Me, @plzombie and @trufanov-nok will oversee the STA fork at organization https://github.com/ImageProcessing-ElectronicPublications. Participation in https://github.com/ScanTailor-Advanced is not required. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much again for this PR!
Wouldn't it be better to join efforts in one repo/organization, instead of all the fragmentation? |
@mara004 say:
Maybe, but not now. I myself specialize in console utilities (GUI is not mine), @plzombie is busy with depress (it is more important to him), @trufanov-nok is in charge of STU. Everyone is busy with their projects. PS: Until I figure out the STA (XML) projects, my help is over. I won't figure it out until I see how it's done. Such are the things. |
Hi @mara004 , @lightsilverberryfox , @vigri . New version? Current: scantailor-advanced/version.h.in Line 6 in b0788fc
|
I've uploaded 1.0.19 now. |
…onicPublications/master feature ScanTailor-Advanced#47: binarizeEdgePlus feature ScanTailor-Advanced#8 feature ScanTailor-Advanced#59
…onicPublications/master feature ScanTailor-Advanced#47: binarizeEdgePlus feature ScanTailor-Advanced#8 feature ScanTailor-Advanced#59
…onicPublications/master feature ScanTailor-Advanced#47: binarizeEdgePlus feature ScanTailor-Advanced#8 feature ScanTailor-Advanced#59
Guys it would be great to have the docs improves to explain the new thresholds, maybe with examples or when to use what. :) |
Hi all, @ireun .
"Recommendations from the author":
|
Hi @vigri and @lightsilverberryfox .
binarizeEdgePlus
into the general functionality. At the same time, I use the Sauvola options, which I don’t think is correct (the new threshold requires smaller area values and larger coefficient values). Due to this, projects with a new threshold are safely saved. Use this PR as a recipe.Good Luck.
Example
Links
Closes #47