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.

Sign up for GitHub

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

Jump to bottom

Rerun tests to update their status. Updated each one to use parametrize. #82

Merged
merged 1 commit into from
Nov 29, 2024
Merged

Rerun tests to update their status. Updated each one to use parametrize. #82

merged 1 commit into from
Nov 29, 2024

Conversation

kmitrovicTT
Copy link
Contributor

@kmitrovicTT kmitrovicTT commented Nov 22, 2024 •
edited
Loading

  • Rerun every test to update their status. Many that were marked as skipped now pass.
  • Updated skip reason for some tests.
  • Removed unused imports
  • Updated tests to use pytest parametrization properly

Before this change (on main): 83 passed, 224 skipped
After: 313 passed, 17 skipped

@kmitrovicTT kmitrovicTT self-assigned this Nov 22, 2024
@kmitrovicTT kmitrovicTT changed the title Rerun tests to update their status. Updated every one to use parametrize. Rerun tests to update their status. Updated each one to use parametrize. Nov 27, 2024
mrakitaTT
mrakitaTT reviewed Nov 27, 2024
View reviewed changes
tests/TTIR/test_array_add.py Outdated Show resolved Hide resolved
tests/TTIR/test_conv2d.py Outdated Show resolved Hide resolved
@kmitrovicTT kmitrovicTT force-pushed the kmitrovic/reorganize_and_update_tests branch 3 times, most recently from 2c6adce to 6f1e3e5 Compare November 28, 2024 14:57
@kmitrovicTT kmitrovicTT force-pushed the kmitrovic/reorganize_and_update_tests branch from 6f1e3e5 to 71475cd Compare November 28, 2024 17:46
@kmitrovicTT kmitrovicTT merged commit b00c49d into main Nov 29, 2024
5 checks passed
@kmitrovicTT kmitrovicTT deleted the kmitrovic/reorganize_and_update_tests branch November 29, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrakitaTT mrakitaTT mrakitaTT approved these changes

@ajakovljevicTT ajakovljevicTT Awaiting requested review from ajakovljevicTT ajakovljevicTT is a code owner

Assignees

@kmitrovicTT kmitrovicTT

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kmitrovicTT @mrakitaTT