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

Increase CI coverage for Gaudi2 #6728

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

raza-sikander
Copy link
Contributor

No description provided.

@loadams
Copy link
Contributor

loadams commented Nov 8, 2024

Tagging @raza-sikander for the failures on the newly added tests. Let us know if we need to modify anything on the node.

@loadams loadams changed the title Increase ci content for Gaudi2 Increase CI coverage for Gaudi2 Nov 11, 2024
@@ -49,51 +49,64 @@ jobs:
TORCHINDUCTOR_COMPILE_THREADS: 1
TEST_LIST: |
test_accelerator.py
test_activation_checkpointing.py
Copy link
Contributor

Choose a reason for hiding this comment

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

@raza-sikander - it looks like the tests pass now but take 2.5 hours to run. Should we switch some of the tests to a cron job if they take that long? Since triggering on as many PRs as this yml file does means we may see queueing there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @loadams, We can split it as you mentioned.
Can you share the process for creation of cron job? So i can split the content two.
And also it would mean we would have two jobs one for CI and other like nightly?

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