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

#12885: Add initial unit tests for infra, specifically for data collection #12933

Merged
merged 13 commits into from
Sep 20, 2024

Conversation

tt-rkim
Copy link
Collaborator

@tt-rkim tt-rkim commented Sep 20, 2024

Ticket

#12885

Problem description

We need more locally-available unit testing in order to validate our infra tools, especially data collection, and to enable more refactoring + feature dev.

What's changed

Add pytests.

Checklist

  • Post commit CI passes
  • Blackhole Post commit (if applicable)
  • Model regression CI testing passes (if applicable)
  • Device performance regression CI testing passes (if applicable)
  • New/Existing tests provide coverage for changes

@tt-rkim tt-rkim merged commit 61329eb into main Sep 20, 2024
7 checks passed
@tt-rkim tt-rkim deleted the rkim/12885-data-tests branch September 20, 2024 14:33
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