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

Skip forge verification #1147

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Skip forge verification #1147

merged 1 commit into from
Feb 6, 2025

Conversation

vbrkicTT
Copy link
Contributor

Ticket

Link to Github Issue

Problem description

Introduced configuration flag for skipping Forge verification to enable an operator test development before an op becomes available in Forge.

What's changed

Introduced new env variable SKIP_FORGE_VERIFICATION.
Optional skipping of Forge module compilation and inference during test verification phase.

Checklist

  • New/Existing tests provide coverage for changes

@vbrkicTT vbrkicTT self-assigned this Jan 30, 2025
@vbrkicTT vbrkicTT force-pushed the vbrkic/skip_forge_verification branch from b117947 to c2469c1 Compare January 30, 2025 18:26
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests505 ran446 passed59 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests564 ran485 passed79 skipped0 failed
TestResult
No test annotations available

@vbrkicTT vbrkicTT marked this pull request as ready for review February 4, 2025 10:44
@vbrkicTT vbrkicTT force-pushed the vbrkic/skip_forge_verification branch from c2469c1 to d6ef36b Compare February 6, 2025 10:39
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests590 ran465 passed125 skipped0 failed
TestResult
No test annotations available

1 similar comment
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests590 ran465 passed125 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests648 ran508 passed140 skipped0 failed
TestResult
No test annotations available

1 similar comment
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests648 ran508 passed140 skipped0 failed
TestResult
No test annotations available

@vbrkicTT vbrkicTT merged commit a5a26a4 into main Feb 6, 2025
10 checks passed
@vbrkicTT vbrkicTT deleted the vbrkic/skip_forge_verification branch February 6, 2025 16:51
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.

2 participants