Skip to content

datafactor: improve opencv (24.0/100, grade F) - #1

Closed
tim777z wants to merge 1 commit into
masterfrom
datafactor/20260913-141457
Closed

tim777z wants to merge 1 commit into
masterfrom
datafactor/20260913-141457

Conversation

@tim777z

@tim777z tim777z commented Sep 13, 2026

Copy link
Copy Markdown
Owner

DataFactor Assessment

Score: 24.0/100 (grade F)

Summary

This is the historical OpenCV monorepo (C++/C/Python/Java), a computer vision library rather than a typical "backend" service; the classifier's label should be read with that caveat. It has real breadth of test files (mostly Java bindings tests and a gtest-based C++ suite) but no CI configuration, no lockfiles/package manifests captured, no lint/format enforcement, and 371 files over 500 LOC (175 over 1000 LOC), much of it vendored third-party code (libtiff, openexr, dshow headers) mixed into the primary source tree. The git history (2015 snapshot, 285 authors, 16949 commits) shows this was once an active large open-source project but the mirrored history in this bundle stops in September 2015, so recency/maintenance cannot be confirmed from evidence.

Recommendations Addressed

  • Make install, build, and test work from a fresh clone [high]
  • Add a docker compose that brings the whole app up [medium]
  • Add CI pipeline running build and tests [high]
  • Make a fresh-clone build and test run self-contained and documented [high]
  • Increase real assertive test coverage instead of stub tests [high]
  • Reduce god files and enforce formatting [medium]
  • Adopt a dependency manifest and vendoring policy for 3rdparty code [medium]

Score: 24.0/100 (grade F)
Source: 2026-09-13T103533-a977e20b05ed4fccb1e1597ef75e1bed.json
@tim777z

tim777z commented Sep 13, 2026

Copy link
Copy Markdown
Owner Author

Closing duplicate - replaced by newer PR

@tim777z tim777z closed this Sep 13, 2026
@tim777z
tim777z deleted the datafactor/20260913-141457 branch September 14, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant