Skip to content

feat[next]: name program and static args in "no program compiled" error - #2734

Open
havogt wants to merge 1 commit into
fix-compiled-program-error-surfacingfrom
improve-compiled-program-miss-message
Open

feat[next]: name program and static args in "no program compiled" error#2734
havogt wants to merge 1 commit into
fix-compiled-program-error-surfacingfrom
improve-compiled-program-miss-message

Conversation

@havogt

@havogt havogt commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Before:

RuntimeError: No program compiled for this set of static arguments.

After:

RuntimeError: No program compiled for this set of static arguments of 'prog': cond=False.
Note that a variant is also selected by the identity of the 'offset_provider' entries and,
for generic programs, by the argument types.

Neither the program nor the offending arguments were named, and the offset provider taking part
in the selection was not mentioned at all, which is the other common cause of an unexpected miss.

The error raised when no compiled variant matches a call only said
"No program compiled for this set of static arguments." — neither which
program nor which arguments. Add both, plus a note that the variant is also
selected by the identity of the 'offset_provider' entries, which is the
other common cause of an unexpected miss.
@havogt
havogt force-pushed the improve-compiled-program-miss-message branch from 84510b2 to 2423589 Compare August 13, 2026 11:12
@havogt
havogt marked this pull request as ready for review August 13, 2026 11:27

@egparedes egparedes 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, just a minor nitpick.

Comment on lines +470 to +471
# Note: not `zip(..., strict=True)`: this runs while building an error message, so a
# mismatch must degrade to a shorter description instead of replacing the error.

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.

I don't get the reference to zip here. Make the comment shorter and mention only the existing code.

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.

2 participants