Skip to content

Comments

WebGPU PR #4 — Interfaces for handling errors and queries#25184

Merged
Elchi3 merged 7 commits intomdn:mainfrom
chrisdavidmills:webgpu-docs-4-errors-queries
Apr 5, 2023
Merged

WebGPU PR #4 — Interfaces for handling errors and queries#25184
Elchi3 merged 7 commits intomdn:mainfrom
chrisdavidmills:webgpu-docs-4-errors-queries

Conversation

@chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Mar 8, 2023

Important: DO NOT merge this PR until #25174 has been merged.

Description

This PR is part of a multi-PR series that aims to document the WebGPU API. The interfaces documented in this PR are:

  • GPUCompilationInfo (done)
  • GPUCompilationMessage (done)
  • GPUDeviceLostInfo (done)
  • GPUError (done)
  • GPUInternalError (done)
  • GPUOutOfMemoryError (done)
  • GPUPipelineError (done)
  • GPUQuerySet (done)
  • GPUUncapturedErrorEvent (done)
  • GPUValidationError (done)

Note that the main PR for this series is #24202 — that one must be merged first, before any others! after that, and each subsequent one, is merged, main should be merged into the next PR, and the documentation integrity must be checked (e.g. do all the links work) before merging.

You can see full details of the additions expected in the WebGPU docs in my research document.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner March 8, 2023 17:48
@chrisdavidmills chrisdavidmills requested review from wbamberg and removed request for a team March 8, 2023 17:48
@github-actions github-actions bot added the Content:WebAPI Web API docs label Mar 8, 2023
@chrisdavidmills chrisdavidmills mentioned this pull request Mar 8, 2023
@chrisdavidmills chrisdavidmills marked this pull request as draft March 8, 2023 17:49
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

Preview URLs (31 pages)
Flaws (7)

Note! 27 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/GPUPipelineError
Title: GPUPipelineError
Flaw count: 1

  • bad_bcd_queries:
    • No BCD data for query: api.GPUPipelineError

URL: /en-US/docs/Web/API/GPUPipelineError/GPUPipelineError
Title: GPUPipelineError()
Flaw count: 1

  • bad_bcd_queries:
    • No BCD data for query: api.GPUPipelineError.GPUPipelineError

URL: /en-US/docs/Web/API/GPUPipelineError/reason
Title: GPUPipelineError.reason
Flaw count: 1

  • bad_bcd_queries:
    • No BCD data for query: api.GPUPipelineError.reason

URL: /en-US/docs/Web/API/GPUQuerySet
Title: GPUQuerySet
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/GPUCommandEncoder/beginRenderPass does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/beginRenderPass does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/beginComputePass does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/writeTimeStamp does not exist
External URLs (14)

URL: /en-US/docs/Web/API/GPUInternalError
Title: GPUInternalError


URL: /en-US/docs/Web/API/GPUInternalError/GPUInternalError
Title: GPUInternalError()


URL: /en-US/docs/Web/API/GPUPipelineError
Title: GPUPipelineError


URL: /en-US/docs/Web/API/GPUPipelineError/GPUPipelineError
Title: GPUPipelineError()


URL: /en-US/docs/Web/API/GPUUncapturedErrorEvent
Title: GPUUncapturedErrorEvent


URL: /en-US/docs/Web/API/GPUUncapturedErrorEvent/GPUUncapturedErrorEvent
Title: GPUUncapturedErrorEvent()


URL: /en-US/docs/Web/API/GPUUncapturedErrorEvent/error
Title: GPUUncapturedErrorEvent.error


URL: /en-US/docs/Web/API/GPUError
Title: GPUError


URL: /en-US/docs/Web/API/GPUError/message
Title: GPUError.message


URL: /en-US/docs/Web/API/GPUCompilationMessage/lineNum
Title: GPUCompilationMessage.lineNum


URL: /en-US/docs/Web/API/GPUValidationError
Title: GPUValidationError


URL: /en-US/docs/Web/API/GPUValidationError/GPUValidationError
Title: GPUValidationError()


URL: /en-US/docs/Web/API/GPUOutOfMemoryError
Title: GPUOutOfMemoryError


URL: /en-US/docs/Web/API/GPUOutOfMemoryError/GPUOutOfMemoryError
Title: GPUOutOfMemoryError()

(comment last updated: 2023-04-05 13:37:15)

Copy link
Contributor

@toji toji left a comment

Choose a reason for hiding this comment

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

Easiest batch yet!

@chrisdavidmills
Copy link
Contributor Author

@toji OK, reference pages for all of the classes in this set now written and ready for review.

@mdn mdn deleted a comment from giftcardvalidator Mar 14, 2023
Copy link
Contributor

@toji toji left a comment

Choose a reason for hiding this comment

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

New additions look good! Two minor clarifications requested regarding queries.

Copy link
Contributor

@toji toji left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@chrisdavidmills chrisdavidmills force-pushed the webgpu-docs-4-errors-queries branch from 716871a to 7dd0ad9 Compare April 5, 2023 13:12
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Merging as discussed in Slack. GPUPipelineError misses compat data, Chris will make a PR to BCD.

@Elchi3 Elchi3 merged commit 93a8dbc into mdn:main Apr 5, 2023
@chrisdavidmills chrisdavidmills deleted the webgpu-docs-4-errors-queries branch April 5, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebAPI Web API docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants