Skip to content

fix: resolve conda_gpu tests - #722

Merged
luisremis merged 15 commits into
developfrom
fix/issue-356
May 24, 2026
Merged

fix: resolve conda_gpu tests#722
luisremis merged 15 commits into
developfrom
fix/issue-356

Conversation

@ad-claw000

Copy link
Copy Markdown
Contributor

Summary

Fixes the underlying issue causing conda_gpu tests to fail by upgrading the docker base image to use Python 3.10.

Verification

  • Checked that pyproject.toml requires python >=3.10
  • Updated base image in docker/pytorch-gpu/Dockerfile
  • Restored the previously removed run_test_conda_gpu step in CI

Fixes #356.

Copilot AI review requested due to automatic review settings May 20, 2026 11:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to fix failing conda_gpu CI tests by moving the GPU test Docker base image to Python 3.10 (matching pyproject.toml’s requires-python >=3.10) and re-enabling the conda GPU test job in the PR workflow.

Changes:

  • Update docker/pytorch-gpu/Dockerfile to use a Python 3.10 GPU base image.
  • Re-add the run_test_conda_gpu GitHub Actions job to build the GPU test image and execute the containerized test suite.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
docker/pytorch-gpu/Dockerfile Switches the GPU test container base image to a Python 3.10 variant.
.github/workflows/pr.yaml Restores a PR CI job that builds/runs GPU conda-based tests in a container.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pr.yaml
Comment thread .github/workflows/pr.yaml
Comment thread .github/workflows/pr.yaml
Comment thread docker/pytorch-gpu/Dockerfile Outdated
Copilot AI review requested due to automatic review settings May 20, 2026 23:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 3 out of 3 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings May 21, 2026 01:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated 3 comments.

Comment thread .github/workflows/pr.yaml
Comment thread docker/pytorch-gpu/build.sh Outdated
Comment thread test/run_test_container.sh Outdated
Copilot AI review requested due to automatic review settings May 21, 2026 04:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread test/run_test_container.sh Outdated
Copilot AI review requested due to automatic review settings May 21, 2026 12:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings May 21, 2026 15:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated no new comments.

@ad-claw000 ad-claw000 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I finally fixed the run_test_conda_gpu failure by explicitly installing openai-clip via git directly in the docker/pytorch-gpu/Dockerfile.

Copilot AI review requested due to automatic review settings May 22, 2026 20:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread docker/pytorch-gpu/Dockerfile Outdated
luisremis
luisremis previously approved these changes May 22, 2026
Copilot AI review requested due to automatic review settings May 23, 2026 09:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 4 changed files in this pull request and generated no new comments.

@luisremis
luisremis merged commit 2800b44 into develop May 24, 2026
7 checks passed
@luisremis
luisremis deleted the fix/issue-356 branch May 24, 2026 00:46
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.

Failing conda_gpu tests

3 participants