Skip to content

Commit 1360dfb

Browse files
authored
Merge pull request #913 from thewtex/json-compatible-data
fix(runPipelineEmscripten): need PipelineInput .data property
2 parents 1b2c131 + 1fff5e2 commit 1360dfb

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/pipeline/internal/runPipelineEmscripten.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ function runPipelineEmscripten (pipelineModule: PipelineEmscriptenModule, args:
9595
}
9696
case InterfaceTypes.JsonCompatible:
9797
{
98-
const dataArray = encoder.encode(JSON.stringify(input))
98+
const dataArray = encoder.encode(JSON.stringify(input.data))
9999
const arrayPtr = setPipelineModuleInputArray(pipelineModule, dataArray, index, 0)
100100
const dataJSON = { size: dataArray.buffer.byteLength, data: `data:application/vnd.itk.address,0:${arrayPtr}` }
101101
setPipelineModuleInputJSON(pipelineModule, dataJSON, index)

test/node/pipeline/runPipelineNodeTest.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ test('runPipelineNode uses input and output json data via memory io', (t) => {
7373
return runPipelineNode(pipelinePath, args, desiredOutputs, inputs)
7474
.then(function ({ outputs }) {
7575
t.is(outputs[0].type, InterfaceTypes.JsonCompatible)
76-
t.deepEqual(outputs[0].data.data, jsonObject)
76+
t.deepEqual(outputs[0].data, jsonObject)
7777
})
7878
})
7979

0 commit comments

Comments
 (0)