Skip to content
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

Bug fix: resample2d output size calculation algorithm and usage #520

Conversation

inexorabletash
Copy link
Member

@inexorabletash inexorabletash commented Jan 19, 2024

  • Per Use deep copy for setting internal slots? #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.


Preview | Diff

index.bs Outdated Show resolved Hide resolved
* Per webmachinelearning#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, and add types.
@inexorabletash inexorabletash force-pushed the bugfix-resample2d-output-sizes branch from 1e6f7e0 to 111172a Compare January 24, 2024 20:20
Copy link
Contributor

@huningxin huningxin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@wchao1115 wchao1115 merged commit 479ce17 into webmachinelearning:main Jan 27, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Jan 27, 2024
SHA: 479ce17
Reason: push, by wchao1115

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@inexorabletash inexorabletash deleted the bugfix-resample2d-output-sizes branch January 27, 2024 18:50
inexorabletash added a commit to inexorabletash/webnn that referenced this pull request Jan 30, 2024
…achinelearning#520)

* Per webmachinelearning#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, and add types.
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.

4 participants