Skip to content

Add CI smoke test (GitHub Actions)#12

Merged
chammoru merged 1 commit into
mainfrom
add-ci
May 31, 2026
Merged

Add CI smoke test (GitHub Actions)#12
chammoru merged 1 commit into
mainfrom
add-ci

Conversation

@chammoru

Copy link
Copy Markdown
Owner

Closes #6.

What

Adds .github/workflows/ci.yml. On push to main and on every PR it:

  1. checks out the repo with Git LFS (so the model weights are materialized),
  2. installs the pinned requirements.txt,
  3. runs predict_cls.py on sample_image/monarch_jpeg_q20.png and asserts the
    predicted quality is within [18, 22]
    (the sample is JPEG quality 20).

The setup reuses classifier/env.sh (PYTHONPATH + TF_USE_LEGACY_KERAS=1), so CI
exercises the exact documented flow.

A CI status badge is added to the README.

Verified locally

  • ci.yml parses as valid YAML; job/steps/triggers checked.
  • The assertion logic was tested against real prediction output: it passes for
    20.01 and correctly fails for an out-of-range value.
  • This PR itself triggers the workflow, so the run on this PR is the live proof.

Add a GitHub Actions workflow that, on push to main and on pull requests:
- checks out the repo with Git LFS (so the model weights are present),
- installs the pinned requirements.txt,
- runs predict_cls.py on the bundled sample image and asserts the predicted
  quality is close to 20 (the sample is JPEG quality 20).

Also add a CI status badge to the README.
@chammoru chammoru merged commit d0c56fd into main May 31, 2026
1 check passed
@chammoru chammoru deleted the add-ci branch May 31, 2026 10:06
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.

[P2] Add a minimal CI smoke test (GitHub Actions)

1 participant