Skip to content

[feat] support SPU link_desc config - #419

Merged
FollyCoolly merged 2 commits into
mainfrom
fix/spu-link-desc-config
Jul 2, 2026
Merged

[feat] support SPU link_desc config#419
FollyCoolly merged 2 commits into
mainfrom
fix/spu-link-desc-config

Conversation

@FollyCoolly

Copy link
Copy Markdown
Contributor

No description provided.

@FollyCoolly
FollyCoolly requested a review from a team as a code owner July 1, 2026 09:02
danqi437
danqi437 previously approved these changes Jul 1, 2026

@danqi437 danqi437 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.

LGTM

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces SPULinkDesc to support custom BRPC link configurations within SPUConfig, allowing per-SPU link descriptions to be merged over default configurations in SPUState. It also updates the runtime cache key to include fxp_fraction_bits and the new BRPC configuration, and adds comprehensive unit tests. The review feedback suggests adding defensive type checks in the from_dict methods of both SPULinkDesc and SPUConfig to gracefully handle malformed inputs and prevent potential AttributeErrors.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread mplang/dialects/spu.py
Comment thread mplang/dialects/spu.py

@danqi437 danqi437 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.

LGTM

@FollyCoolly
FollyCoolly merged commit 93678b1 into main Jul 2, 2026
11 checks passed
@FollyCoolly
FollyCoolly deleted the fix/spu-link-desc-config branch July 2, 2026 02:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants