Skip to content

[TESTS] Fix failing nightly pytorch model tests#34684

Merged
mvafin merged 1 commit intoopenvinotoolkit:masterfrom
mvafin:mvafin/pt_fe/nightly_fix
Mar 13, 2026
Merged

[TESTS] Fix failing nightly pytorch model tests#34684
mvafin merged 1 commit intoopenvinotoolkit:masterfrom
mvafin:mvafin/pt_fe/nightly_fix

Conversation

@mvafin
Copy link
Contributor

@mvafin mvafin commented Mar 13, 2026

Details:

  • Fix failing nightly pytorch model tests

Tickets:

  • ticket-id

AI Assistance:

  • AI assistance used: no / yes
  • If yes, summarize how AI was used and what human validation was performed (build/tests/manual checks).

@mvafin mvafin requested a review from a team as a code owner March 13, 2026 12:10
@github-actions github-actions bot added the category: PyTorch FE OpenVINO PyTorch Frontend label Mar 13, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

subprocess.run([sys.executable, "-m", "pip", "install",
"git+https://github.com/facebookresearch/detectron2.git@017abbfa5f2c2a2afa045200c2af9ccf2fc6227f"])

@retry(3, exceptions=(OSError,), delay=10, exponential_backoff=True, backoff_multiplier=2, max_delay=120)
Copy link
Contributor

Choose a reason for hiding this comment

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

🫣

f"git clone https://github.com/pytorch/benchmark.git {self.repo_dir.name}")
subprocess.check_call(
["git", "checkout", "dbc109791dbb0dfb58775a5dc284fc2c3996cb30"], cwd=self.repo_dir.name)
["git", "checkout", "364420aeca07d9519840a5b6e771035e4dff9d72"], cwd=self.repo_dir.name)
Copy link
Contributor

Choose a reason for hiding this comment

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

I have a feeling, that this should happen on the buildsystem level. Probably a change for a different delivery though.

@mvafin mvafin added this pull request to the merge queue Mar 13, 2026
Merged via the queue into openvinotoolkit:master with commit edde158 Mar 13, 2026
209 of 210 checks passed
@mvafin mvafin deleted the mvafin/pt_fe/nightly_fix branch March 13, 2026 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: PyTorch FE OpenVINO PyTorch Frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants