abrown edited PR #8790.
rahulchaphalkar updated PR #8790.
rahulchaphalkar updated PR #8790.
rahulchaphalkar commented on PR #8790:
@abrown I pushed some changes to fix failures due to some incompatibilities with
Id
enum. Tests are now passing, but vet step is failing, maybe due to one of my rebases. Can you vet the dependency? Thanks.
I already did... where _did_ that commit go?
abrown updated PR #8790.
abrown submitted PR review.
abrown submitted PR review.
abrown created PR review comment:
If possible, let's rewrite this as
impl From<ExecutionTarget> for DeviceType<'static>
. If not possible then at least change the function name and documentation.
abrown created PR review comment:
I think this could get cleaned up to something like:
ElementType::Undefined | ElementType::Dynamic | ... => Err(BackendError::SomeNewVariant("{:?} is not yet supported in the openvino backend", element_type))
abrown created PR review comment:
This can be a
TryFrom
since it is more useful to return an error to the user than crash Wasmtime withtodo!
.
abrown created PR review comment:
Similarly, this could be
impl From<TensorType> for ElementType
.
lum1n0us submitted PR review.
lum1n0us created PR review comment:
If no need to specify layout (of input tensor), and there is no PrePostProcessing in the whole file, Does it mean wasm code has to make sure the input tensor matches the requirement of model input? (I assume openvino models requires "NHWC" input)
rahulchaphalkar updated PR #8790.
rahulchaphalkar submitted PR review.
rahulchaphalkar created PR review comment:
Hi Liang, yes this requires input tensor layout match the expected layout for model input.
abrown submitted PR review.
abrown merged PR #8790.
Last updated: Dec 23 2024 at 12:05 UTC