Skip to content

Commit 4cca9c4

Browse files
Update base for 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]
1 parent 6bdca9c commit 4cca9c4

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

devtools/bundled_program/bundled_program.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,9 @@ ET_NODISCARD Error load_bundled_input(
267267
"testset_idx %zu is out of range [0, %u]",
268268
testset_idx,
269269
test_cases->size());
270-
auto bundled_inputs = test_cases->Get(static_cast<flatbuffers::uoffset_t>(testset_idx))->inputs();
270+
auto bundled_inputs =
271+
test_cases->Get(static_cast<flatbuffers::uoffset_t>(testset_idx))
272+
->inputs();
271273

272274
for (size_t input_idx = 0; input_idx < method.inputs_size(); input_idx++) {
273275
auto bundled_input = bundled_inputs->GetMutableObject(input_idx);
@@ -372,7 +374,8 @@ ET_NODISCARD Error verify_method_outputs(
372374
testset_idx,
373375
test_cases->size());
374376
auto bundled_expected_outputs =
375-
test_cases->Get(static_cast<flatbuffers::uoffset_t>(testset_idx))->expected_outputs();
377+
test_cases->Get(static_cast<flatbuffers::uoffset_t>(testset_idx))
378+
->expected_outputs();
376379

377380
if (bundled_expected_outputs->size() == 0) {
378381
// No bundled expected outputs, so we can't verify the method outputs.

0 commit comments

Comments
 (0)