The Wayback Machine - https://web.archive.org/web/20221109074726/https://github.com/neuralmagic/deepsparse/pull/379
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add smoke test GitHub Action workflow #379

Merged
merged 10 commits into from May 26, 2022
Merged

Conversation

dbarbuzzi
Copy link
Contributor

@dbarbuzzi dbarbuzzi commented May 20, 2022

The main goal of this PR is to add a testing workflow to the project’s GitHub Actions to run a set of smoke tests during PRs targeted to main and release branches. Changes include:

  • Adding a pytest smoke marker to a variety of tests to be included
    • Base: all
    • CLI: All commands with a --help flag, one or more of each command for a reasonable breadth of coverage
    • Examples: All instances tests where batch size was 1 or 8
  • Updating the Makefile to add a target which excludes base tests to better parallelize test runs without overlap
  • Create a new GitHub Action workflow to run the base, CLI, and example smoke tests in parallel

@dbarbuzzi dbarbuzzi requested review from mgoin, bfineran, markurtz, dhuangnm and a team May 20, 2022
@dbarbuzzi dbarbuzzi self-assigned this May 20, 2022
mgoin
mgoin approved these changes May 26, 2022
@dbarbuzzi dbarbuzzi merged commit ffe6d61 into main May 26, 2022
4 checks passed
@dbarbuzzi dbarbuzzi deleted the add-test-check-workflow branch May 26, 2022
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.

None yet

3 participants