-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SuperResolution example and test #284
Conversation
a62e020
to
80aae58
Compare
@fujunwei PTAL! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. LGTM and left some comments.
Could you please add a ReadMe for the example like [1]?
[1] https://github.com/webmachinelearning/webnn-native/blob/main/examples/MobileNetV2/README.md
return utils::BuildConstant(builder, data.shape, data.data<float>(), data.num_bytes()); | ||
} | ||
|
||
const wnn::Operand SuperResolution::BuildConv(const wnn::GraphBuilder& builder, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mingmingtasd Ditto.
Adds SuperResolution to WebNN!
80aae58
to
7c4030c
Compare
@fujunwei I don't have the example building yet (for the README), only the test. Will come next if that's OK. |
make sense, merge it, thanks. |
Adds SuperResolution to WebNN!