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

Stop tracking of 2/3 of target models due to CI limitations #1044

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

nvukobratTT
Copy link
Contributor

Due to our current CI limitations, our nightly jobs are mostly broken. As broken nightly isn't providing any value, our best effort is to lower the number of models we're targeting and focus on solving CI limitations.

Main target should be to limit the number of variants and still keep plenty of unique architectures. That way, we'll not lower our model coverage quality even with 2/3 of variants removed as a target.

In sum, majority of CI limitations come from limited disk space, host DRAM, etc.

Fix #1043

Due to our current CI limitations, our nightly jobs are mostly broken.
As broken nightly isn't providing any value, our best effort is to lower
the number of models we're targeting and focus on solving CI
limitations.

Main target should be to limit the number of variants and still keep
plenty of unique architectures. That way, we'll not lower our model
coverage quality even with 2/3 of variants removed as a target.

In sum, majority of CI limitations come from limited disk space, host
DRAM, etc.

Fix #1043
Copy link
Contributor

@dgolubovicTT dgolubovicTT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems ok

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests691 ran429 passed262 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests760 ran489 passed271 skipped0 failed
TestResult
No test annotations available

@nvukobratTT nvukobratTT merged commit 45a03b8 into main Jan 14, 2025
9 checks passed
@nvukobratTT nvukobratTT deleted the nvukobrat/skip_model_tests_due_to_ci_limitations branch January 14, 2025 16:13
chandrasekaranpradeep added a commit that referenced this pull request Jan 17, 2025
…method in model analysis (#1060)

In [this PR](#1044), we
skipped some of the model variants due to CI limitation and those model
variants will also be skipped in the model analysis pipeline. To avoid
skipping the model variants in model analysis, added `--no-skips`
argument in the subprocess run method in extraction and generation of
unique ops config test function(i.e
`generate_and_export_unique_ops_tests in
scripts/model_analysis/unique_ops_utils.py`)
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.

Stop tracking of 2/3 of target models due to CI limitations
4 participants