OMRChecker v2.0.0#220
Conversation
…eature/colored-evaluation-outputs
#250) * refactor(preprocessors): replace hardcoded values with config/CLI parameters * refactor: make erosion kernel a tuple and rename threshold variables for clarity * fix: rename threshold variables and remove unnecessary __all__ list --------- Co-authored-by: Veda Vyasa <2400040217@kluniversity.in>
…verridden timezone env
fix: update snapshot
|
Thanks! The updates look great. Is this new version ready to test / use? Do you have an idea when they will be merged? @paolini and I have been using v1 for a project related to the Italian math Olympiad; it would be great to have all these new features in main (and maybe we can contribute some code back to the project once we're using v2). |
|
@fph thanks for checking in. There are active efforts going on with more streamlined approach for the code to be maintainable. I'm simplifying the processing pipeline and also preparing a browser-only version in the latest branch Cannot provide a timeline on merging to master since it's a side project. But I can make it stable to be usable for your usecase. You may text me over discord channel and I'll be happy to unblock you in using it. |
Kept dev branch architecture (src.algorithm.*, src.processors.internal.*, src.schemas.*) over master's PR #250 constants refactor which was based on an older codebase. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
uv sync --frozen already creates the .venv; calling uv venv again fails with "virtual environment already exists". source activate also has no effect between steps — uv run handles venv activation itself. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
CI Feedback 🧐(Feedback updated until commit 0f453e0)A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
Closes multiple feature requests/issues: #156 #128 #125 #105 #39 #185 #247
Changelogs from #237
Changelogs from #186
Changelogs from #192
Fixes #190 and #191
Also addresses: #36
Other Features covered
feat: template roi gui; created local scripts folder as todo; add new template for conditional sets
fix: defaultSelector issue
feat: support for customFieldTypes in template.json; fix template matching issue
Added support for show_preprocessor_diff{} to display per-processor level before-after images (apart from show_image_level)
Stabilised autorotate by giving option to crop from preview
preview can be enabled for all preprocessors in config.json (autorotating sample is available in Autorotating folder in samples)
Changelogs from #197
Current Master branch:
~35 Files, 4441 loc == Avg 126.8 lines per file, yet highly coupled code and restricted to bubble fields.
After code merge
~112 Files, 16000 loc == Avg 142.8 lines per file, Yet much more readable, more extensible and full of features.