Skip to content

Conversation

@polarG
Copy link
Contributor

@polarG polarG commented Jan 6, 2026

Description

  • When building the CUDA 11.1.1 image, pip (Python 3.8) cannot find a pre-built wheel for the latest wandb release (v0.23.1). As a result, pip attempts to build wandb from source. However, the build fails because the image does not have Go installed, which is required for building wandb from source. Then the error appears.

Solution

  • For the CUDA 11.1.1 build, install the required build tools (e.g., Go, Rust, and Cargo) needed for wandb.

@polarG polarG self-assigned this Jan 6, 2026
@polarG polarG added the enhancement New feature or request label Jan 6, 2026
@polarG polarG requested a review from a team as a code owner January 6, 2026 22:55
@polarG polarG added CI/CD Continuous integration or deployment dependencies Pull requests that update a dependency file labels Jan 6, 2026
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.70%. Comparing base (c99380b) to head (abaf410).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #771   +/-   ##
=======================================
  Coverage   85.69%   85.70%           
=======================================
  Files         102      102           
  Lines        7699     7703    +4     
=======================================
+ Hits         6598     6602    +4     
  Misses       1101     1101           
Flag Coverage Δ
cpu-python3.10-unit-test 70.96% <77.77%> (-0.02%) ⬇️
cpu-python3.12-unit-test 70.96% <77.77%> (+0.01%) ⬆️
cpu-python3.7-unit-test 70.43% <100.00%> (+0.01%) ⬆️
cuda-unit-test 83.59% <77.77%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@polarG polarG changed the title CI/CD - Fix Image build for cuda11.1.1 CI/CD - Fix Image build for cuda11.1.1 and update numpy API Jan 7, 2026
@polarG polarG requested review from abuccts and guoshzhao January 7, 2026 18:33
Copy link
Member

@abuccts abuccts left a comment

Choose a reason for hiding this comment

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

pip (Python 3.8) cannot find a pre-built wheel for the latest wandb release (v0.23.1)

According to wandb v0.23.1, it's compatible with py3.8. Maybe you need to upgrade pip/wheel?

However, the build fails because the image does not have Go installed, which is required for building wandb from source.

Why not just add an apt-get install -y golang?

@polarG polarG changed the title CI/CD - Fix Image build for cuda11.1.1 and update numpy API CI/CD - Remove Image build for cuda11.1.1 and update numpy API Jan 12, 2026
@polarG polarG enabled auto-merge (squash) January 12, 2026 22:25
@polarG
Copy link
Contributor Author

polarG commented Jan 12, 2026

pip (Python 3.8) cannot find a pre-built wheel for the latest wandb release (v0.23.1)

According to wandb v0.23.1, it's compatible with py3.8. Maybe you need to upgrade pip/wheel?

However, the build fails because the image does not have Go installed, which is required for building wandb from source.

Why not just add an apt-get install -y golang?

Added build-essential packages to the CUDA 11.1.1 Dockerfile to enable building wandb.

Hongtao Zhang added 6 commits January 13, 2026 18:08
This reverts commit 059daa9.

	modified:   .github/workflows/build-image.yml
	new file:   dockerfile/cuda11.1.1.dockerfile
This reverts commit 4f23173.

	modified:   dockerfile/cuda11.1.1.dockerfile
@polarG polarG changed the title CI/CD - Remove Image build for cuda11.1.1 and update numpy API CI/CD - Fix Image build for cuda11.1.1 and unit test pipelines Jan 14, 2026
@polarG polarG changed the title CI/CD - Fix Image build for cuda11.1.1 and unit test pipelines CI/CD - Fix Image build for cuda11.1.1 and py unit test pipelines Jan 14, 2026
@abuccts abuccts requested a review from Copilot January 14, 2026 23:30
Copilot AI review requested due to automatic review settings January 21, 2026 23:08
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 1 out of 1 changed files in this pull request and generated 6 comments.


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

Copy link
Member

@abuccts abuccts left a comment

Choose a reason for hiding this comment

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

Can you update the title and descriptions accordingly?

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


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

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


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

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


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

**Description**
- Deprecated API arguments, 'interpolation', when numpy >v1.22.0. And it
was removed since v2.4.0.
- Pandas to_exel() failed in unit test.

Solution
- Update the NumPy API usage to support NumPy 1.22.0+ while maintaining
backward compatibility with earlier versions.
- Fixed the pandas to_excel() API compatibility issue. In newer versions
of pandas (2.0+), the 'sheet_name' parameter must be passed as a keyword
argument rather than a positional argument

---------

Co-authored-by: Hongtao Zhang <hongtaozhang@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@polarG polarG changed the title CI/CD - Fix Image build for cuda11.1.1 and py unit test pipelines CI/CD - Fix Image build for cuda11.1.1 Jan 23, 2026
@polarG
Copy link
Contributor Author

polarG commented Jan 23, 2026

/azp run CodeQL

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@polarG
Copy link
Contributor Author

polarG commented Jan 23, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@polarG
Copy link
Contributor Author

polarG commented Jan 23, 2026

/retest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Continuous integration or deployment dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants