Skip to content

PyTorch with Kineto Spyre Build Script Fix - #30

Open
tej7499 wants to merge 2 commits into
mainfrom
build-script-change
Open

PyTorch with Kineto Spyre Build Script Fix #30
tej7499 wants to merge 2 commits into
mainfrom
build-script-change

Conversation

@tej7499

@tej7499 tej7499 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator
  1. Fix Pytorch with kineto-spyre build for non x86 archs.
  2. Added platform tag to the generated wheel.

tej7499 added 2 commits June 26, 2026 15:27
…tags to wheels

Signed-off-by: Tejas Holla <Tejas.Holla@ibm.com>
Signed-off-by: Tejas Holla <Tejas.Holla@ibm.com>
@tej7499
tej7499 requested review from lasch and yuhaohaoyu June 26, 2026 10:11
@lasch

lasch commented Jun 29, 2026

Copy link
Copy Markdown
Member

There's a build issue with this PR. Is that a problem caused by the PR itself?
Otherwise: lgtm

@tej7499

tej7499 commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator Author

There's a build issue with this PR. Is that a problem caused by the PR itself? Otherwise: lgtm

The problem is not caused by the PR. The build failure is seen in previous merged PR pipeline logs as well.

@johnuman

Copy link
Copy Markdown

@lasch @yuhaohaoyu @tej7499

I think this PR does not cover the different arch been used to build kineto-spyre.

This is the script I used to build wheels on different archs (x86 and s390x) for regular wheel and manylinux wheel.

See attached build script
build_pytorch_v2.11.sh
build_pytorch_manylinux_v2.11.sh

@tej7499

tej7499 commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator Author

@johnuman, These changes worked for x86 and Power for creating kineto-spyre wheels using PyTorch 2.11. Your scripts are more comprehensive covering both the regular and manylinux wheels. Will check on Power by creating the PyTorch 2.12 kineto-spyre wheel and see if everything is good. We can then close this PR.

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.

4 participants