Skip to content

Conversation

@ProExpertProg
Copy link
Collaborator

@ProExpertProg ProExpertProg commented Oct 13, 2025

In-progress PR to test 2.9 in CI with inductor partitioning and standalone compile.

Includes:

Fixes in main:

Past fixes no longer necessary:

Signed-off-by: Huy Do <[email protected]>
Signed-off-by: Huy Do <[email protected]>
Signed-off-by: Huy Do <[email protected]>
Signed-off-by: Huy Do <[email protected]>
Signed-off-by: Huy Do <[email protected]>
@ProExpertProg ProExpertProg changed the title [DO NOT MERGE] 2.9, Inductor partition unit tests, monkeypatch fix [DO NOT MERGE] 2.9, Inductor partition, standalone compile, monkeypatch fix(es) Oct 15, 2025
@ProExpertProg
Copy link
Collaborator Author

@BoyuanFeng LORA test failure seems relevant: perhaps we need to review how we deal with weak_ref_tensors with inductor partitioning but also not sure why there's a CPU tensor there.

@ProExpertProg ProExpertProg force-pushed the luka/2.9-inductor-partition-test-with-fix branch from e811cb5 to b4518de Compare October 15, 2025 15:43
@mergify mergify bot removed the needs-rebase label Oct 15, 2025
@ProExpertProg ProExpertProg force-pushed the luka/2.9-inductor-partition-test-with-fix branch from b4518de to 6f1222c Compare October 16, 2025 00:41
@mergify
Copy link

mergify bot commented Oct 16, 2025

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @ProExpertProg.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify bot added the needs-rebase label Oct 16, 2025
@ProExpertProg ProExpertProg force-pushed the luka/2.9-inductor-partition-test-with-fix branch from 47bbdff to 06601b7 Compare October 16, 2025 20:36
@mergify
Copy link

mergify bot commented Oct 16, 2025

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @ProExpertProg.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@ProExpertProg ProExpertProg force-pushed the luka/2.9-inductor-partition-test-with-fix branch from 06601b7 to f6b27e6 Compare October 17, 2025 14:32
@mergify mergify bot removed the needs-rebase label Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build documentation Improvements or additions to documentation llama Related to Llama models ready ONLY add when PR is ready to merge/full CI is needed rocm Related to AMD ROCm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants