Skip to content

Commit cea7e74

Browse files
author
zhenyanzhang
committed
Update on "[ExecuTorch][#10447] Extend PyBundledModule with extension.BundledModule"
#10447 # Context This issue is a step of #9638. In #9638, we want to have `extension.Module` as the single source of implementation in `pybindings`, which means that `pybindings.PyModule` should use `extension.Module` rather than its own `pybindings.Module`. # Proposal Now that we have `extension.BundledModule` ready, we want to test it out by having our existing `PyBundledModule` to extend it, and let `verify_result_with_bundled_expected_output` to use it, so that we can test out the whole thing with https://github.com/pytorch/executorch/blob/fb45e19055a92d2a91a4d4b7008e135232cbb14b/devtools/bundled_program/test/test_end2end.py Differential Revision: [D73564127](https://our.internmc.facebook.com/intern/diff/D73564127/) [ghstack-poisoned]
2 parents fc541bb + 6116b81 commit cea7e74

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

extension/pybindings/pybindings.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -852,7 +852,7 @@ struct PyModule final {
852852

853853
THROW_IF_ERROR(
854854
outputs.error(),
855-
"load_bundled_input failed with status 0x%" PRIx32,
855+
"Execution failed with status 0x%" PRIx32,
856856
static_cast<uint32_t>(outputs.error()));
857857

858858
status = m.verify_method_outputs(method_name, testset_idx, rtol, atol);

0 commit comments

Comments
 (0)