You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@hamilton.apache.org by GitBox <gi...@apache.org> on 2026/07/31 20:48:36 UTC

[GH] (hamilton/fix/materialize-unbound-function-graph): Workflow run "Unit Tests" failed!

The GitHub Actions job "Unit Tests" on hamilton.git/fix/materialize-unbound-function-graph has failed.
Run started by GitHub user chuenchen309 (triggered by skrawcz).

Head commit for run:
929a647e328ef9c18cb51a0acc82615685e5ce5a / chuenchen309 <48...@users.noreply.github.com>
Fix UnboundLocalError masking real errors in Driver.materialize()

The `finally` block at driver.py:1616 passes `function_graph` to the
post_graph_execute hook, but that local only binds at :1568. Any error
raised before it -- including the "No output requested" ValueError added
in #810 for #796 -- makes the finally die with UnboundLocalError instead,
hiding the real cause.

`outputs` and `materializer_vars` were already pre-initialized for exactly
this reason ("This is so the finally logging statement does not accidentally
die"); `function_graph` was missed. Pre-initialize it the same way.

This only surfaces when a post_graph_execute hook is registered, which
gates the finally block. #796's reporter used h_experiments.ExperimentTracker
-- one such hook -- so the fix in #810 never reached the reported case.

Co-Authored-By: Claude Opus 4.8 <no...@anthropic.com>

Report URL: https://github.com/apache/hamilton/actions/runs/29581092212

With regards,
GitHub Actions via GitBox