Skip to content

datafactor: improve seaborn (32.4/100, grade F) - #1

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

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

Conversation

@tim777z

@tim777z tim777z commented Sep 13, 2026

Copy link
Copy Markdown
Owner

DataFactor Assessment

Score: 32.4/100 (grade F)

Summary

Seaborn (as captured in this repository snapshot, 2015 vintage, v0.7.0.dev) is a mature, well-tested statistical plotting library, not really an "ai_research" project despite the classifier's label. It has a real test suite (10 spec files, 1:7 ratio) exercising public plotting API behavior, decent docstrings/API docs embedded in source, and Travis CI, but no coverage enforcement, no lint/type enforcement in CI, several 900-2000+ LOC god files, no lockfile, no Dockerfile or reproducibility tooling, and no experiment-tracking or ML-specific reproducibility apparatus since this is a visualization library rather than an ML/research codebase. Onboarding is functional (README covers install/test) but thin on architecture, changelog, and environment reproducibility.

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 an enforced test job to CI that actually runs the suite [high]
  • Introduce a lockfile and pin dependency versions [medium]
  • Split god files and add lint enforcement [medium]
  • Document architecture and add reproducible dev environment [medium]
  • Sustain small, test-paired commits going forward [high]

Score: 32.4/100 (grade F)
Source: 2026-09-13T104536-e2b6a10043a748e69a78e738cdd42f38.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-141705 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