Skip to content

Commit 1e6f7e0

Browse files
Bug fix: resample2d output size calculation algorithm and usage
* Per #395 make it clearer that the algorithm is working on a new descriptor (a copy of the input's) * When invoking the algorithm, pass the required input argument. * Rename algorithm from "resample output sizes" to "calculate resample output sizes" which more clearly descripts what is happening and aligns with similar algorithms in the spec.
1 parent 50a00b9 commit 1e6f7e0

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

index.bs

+3-4
Original file line numberDiff line numberDiff line change
@@ -5278,10 +5278,10 @@ partial interface MLGraphBuilder {
52785278
<details open algorithm>
52795279

52805280
<summary>
5281-
To <dfn for="MLGraphBuilder">resample output sizes</dfn> given |input| and |options|, run the following steps:
5281+
To <dfn for="MLGraphBuilder">calculate resample output sizes</dfn> given |input| and |options|, run the following steps:
52825282
</summary>
52835283
<div class=algorithm-steps>
5284-
1. Let |desc| be an {{MLOperandDescriptor}} initialized to |input|.{{MLOperand/[[descriptor]]}}.
5284+
1. Let |desc| be a new {{MLOperandDescriptor}} initialized to |input|.{{MLOperand/[[descriptor]]}}.
52855285
1. For |index| in [=the range=] 0 to the [=list/size=] of |options|.{{MLResample2dOptions/axes}}, exclusive:
52865286
1. If |options|.{{MLResample2dOptions/sizes}} [=map/exists=], set |desc|.{{MLOperand/[[descriptor]]}}.{{MLOperandDescriptor/dimensions}}[|options|.{{MLResample2dOptions/axes}}[|index|]] to |options|.{{MLResample2dOptions/sizes}}[|index|] and return |desc|.
52875287
1. Otherwise, set |desc|.{{MLOperand/[[descriptor]]}}.{{MLOperandDescriptor/dimensions}}[|options|.{{MLResample2dOptions/axes}}[|index|]] to |input|.{{MLOperand/[[descriptor]]}}.{{MLOperandDescriptor/dimensions}}[|index|] multiplied by |options|.{{MLResample2dOptions/scales}}.
@@ -5297,8 +5297,7 @@ partial interface MLGraphBuilder {
52975297
<div class=algorithm-steps>
52985298
1. If the [=list/size=] of |input|.{{MLOperand/[[descriptor]]}}.{{MLOperandDescriptor/dimensions}} is not 4, then [=exception/throw=] a "{{DataError}}" {{DOMException}}.
52995299
1. If [=checking resample options=] given |options| returns false, then [=exception/throw=] a "{{DataError}}" {{DOMException}}.
5300-
1. Let |desc| be the result of [=resampling output sizes=] given |options|.
5301-
1. If that [=exception/throws=] an error, re-[=exception/throw=] the error.
5300+
1. Let |desc| be the result of [=calculating resample output sizes=] given |input| and |options|.
53025301
1. If any of the following sub-steps fail, [=exception/throw=] an "{{OperationError}}" {{DOMException}}.
53035302
1. Let |output| be the result of [=creating an MLOperand=] given [=this=] and |desc|.
53045303
1. Make a request to the underlying platform to:

0 commit comments

Comments
 (0)