Skip to content

Add an upsample2d operator as experimental#180

Merged
houseroad merged 1 commit intoonnx:masterfrom
houseroad:resizenearest
Nov 14, 2017
Merged

Add an upsample2d operator as experimental#180
houseroad merged 1 commit intoonnx:masterfrom
houseroad:resizenearest

Conversation

@houseroad
Copy link
Copy Markdown
Member

This is experimental.

@houseroad houseroad requested review from bddppq and jamesr66a November 2, 2017 19:42
Comment thread onnx/defs/experiments/defs.cc Outdated
.Attr(
"width_scale",
"The scale along width dimension",
AttrType::FLOAT)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please either mark the attribute as required or document the default value.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure

"Y",
"4D tensor after resizing, N x C x H x W")
.SetDoc(R"DOC(
Resize the width and height dimensions:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add some more description? It's not clear what values of the new pixels in the output.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will add more description, and an example here.

@bddppq
Copy link
Copy Markdown
Contributor

bddppq commented Nov 14, 2017

LGTM

@ebarsoum

@bddppq bddppq self-requested a review November 14, 2017 22:58
@houseroad houseroad merged commit 3cb6e28 into onnx:master Nov 14, 2017
@silaskuo
Copy link
Copy Markdown

Hello, do you have ETA to move this op from experimental to default. Thanks

@houseroad
Copy link
Copy Markdown
Member Author

@silaskuo right on time, I will move it soon. PR is under construction.

@silaskuo
Copy link
Copy Markdown

Thanks for the quick reply. Look forward to it!

@silaskuo
Copy link
Copy Markdown

Is it moved from experimental to default? If yes, then is https://github.com/onnx/onnx/blob/master/docs/Operators.md going to be updated accordingly? Thanks.

jfnhiller pushed a commit to jfnhiller/onnx that referenced this pull request Feb 26, 2026
add link to backend-scoreboard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants