Skip to content

Implement CI, tests, and update documentation#2

Merged
KnightKotOR merged 8 commits into
mainfrom
feature/testing
Jun 28, 2026
Merged

Implement CI, tests, and update documentation#2
KnightKotOR merged 8 commits into
mainfrom
feature/testing

Conversation

@KnightKotOR

Copy link
Copy Markdown
Owner

Linter:

  • Added Ruff for formatting and linting

Tests:

  • Data: Validate data columns and classes balance
  • Smoke: Validate imports and inspect signature
  • Metrics: Validate F1-macro threshold above 0.85 and no High → Low predictions
  • Optuna objective: Validate creation, optimizing study and raise on unknown model
  • Invariants: Validate labels (0,1,2); Validate High priority tickets from test either skip if none; Validate that prediction is deterministic

CI:

  • On push and pull request into main testing lint and pytests

Docs:

  • Updated readme and added pictures for confusion matrix and shap result

Refactoring:

  • Added assets folder for readme pictures

fix: updated pyproject.toml
refactor: better imports and formatting in optuna_search.py
style: formatting and code style update in support_tickets.ipynb
chore: uv.lock now has ruff dependency
test (conftest): created fixtures - dataframe, model, high priority ticket
test (schema): created tests for data
test (smoke): created tests for imports
chore (uv): added libraries for testing
docs (notebook): fixed correlation value in a comment from 0.89 to 0.94
fix: added train and test separate fixtures
fix: changed high_priority_ticket values to a sample from the test dataset to avoid testing on a ticket from the train
@KnightKotOR KnightKotOR merged commit b16ad79 into main Jun 28, 2026
8 checks passed
@KnightKotOR KnightKotOR deleted the feature/testing branch July 4, 2026 15:07
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