Skip to content

Comments

WebGPU PR #2 — Interfaces for representing pipeline resources#25140

Merged
Elchi3 merged 15 commits intomdn:mainfrom
chrisdavidmills:webgpu-docs-2-resources
Apr 5, 2023
Merged

WebGPU PR #2 — Interfaces for representing pipeline resources#25140
Elchi3 merged 15 commits intomdn:mainfrom
chrisdavidmills:webgpu-docs-2-resources

Conversation

@chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Mar 7, 2023

Important: DO NOT merge this PR until #24202 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:

  • GPUBuffer (done)
  • GPUExternalTexture (done)
  • GPUSampler (done)
  • GPUShaderModule (done)
  • GPUTexture (done)
  • GPUTextureView (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 7, 2023 10:36
@chrisdavidmills chrisdavidmills requested review from wbamberg and removed request for a team March 7, 2023 10:36
@github-actions github-actions bot added the Content:WebAPI Web API docs label Mar 7, 2023
@chrisdavidmills chrisdavidmills marked this pull request as draft March 7, 2023 10:36
@chrisdavidmills chrisdavidmills mentioned this pull request Mar 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2023

Preview URLs (31 pages)
Flaws (42)

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

URL: /en-US/docs/Web/API/GPUExternalTexture
Title: GPUExternalTexture
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist
    • /en-US/docs/Web/API/GPUBindGroup does not exist

URL: /en-US/docs/Web/API/GPUExternalTexture/label
Title: GPUExternalTexture.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

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

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUBuffer/mapAsync
Title: GPUBuffer.mapAsync()
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUValidationError does not exist

URL: /en-US/docs/Web/API/GPUBuffer/label
Title: GPUBuffer.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUBuffer/usage
Title: GPUBuffer.usage
Flaw count: 11

  • macros:
    • /en-US/docs/Web/API/GPUCommandEncoder/copyBufferToBuffer does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/copyTextureToBuffer does not exist
    • /en-US/docs/Web/API/GPURenderPassEncoder/setIndexBuffer does not exist
    • /en-US/docs/Web/API/GPURenderPassEncoder/drawIndirect does not exist
    • /en-US/docs/Web/API/GPURenderPassEncoder/dispatchWorkgroupsIndirect does not exist
    • and 6 more flaws omitted

URL: /en-US/docs/Web/API/GPUShaderModule
Title: GPUShaderModule
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist
    • /en-US/docs/Web/API/GPUCompilationInfo does not exist

URL: /en-US/docs/Web/API/GPUShaderModule/label
Title: GPUShaderModule.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUShaderModule/getCompilationInfo
Title: GPUShaderModule.getCompilationInfo()
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/GPUCompilationInfo does not exist
    • /en-US/docs/Web/API/GPUCompilationInfo does not exist
    • /en-US/docs/Web/API/GPUCompilationInfo does not exist
    • /en-US/docs/Web/API/GPUCompilationMessage does not exist

URL: /en-US/docs/Web/API/GPUTextureView
Title: GPUTextureView
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/beginRenderPass does not exist

URL: /en-US/docs/Web/API/GPUTextureView/label
Title: GPUTextureView.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

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

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUSampler/label
Title: GPUSampler.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUTexture
Title: GPUTexture
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist
    • /en-US/docs/Web/API/GPUQueue/copyExternalImageToTexture does not exist

URL: /en-US/docs/Web/API/GPUTexture/label
Title: GPUTexture.label
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist

URL: /en-US/docs/Web/API/GPUTexture/usage
Title: GPUTexture.usage
Flaw count: 7

  • macros:
    • /en-US/docs/Web/API/GPUCommandEncoder/copyTextureToBuffer does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/copyBufferToTexture does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/beginRenderPass does not exist
    • /en-US/docs/Web/API/GPUBindGroup does not exist
    • /en-US/docs/Web/API/GPUBindGroupLayout does not exist
    • and 2 more flaws omitted

URL: /en-US/docs/Web/API/GPUTexture/createView
Title: GPUTexture.createView()
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/GPUError does not exist
    • /en-US/docs/Web/API/GPUValidationError does not exist
    • /en-US/docs/Web/API/GPUCommandEncoder/beginRenderPass does not exist
External URLs (12)

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


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


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


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


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


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


URL: /en-US/docs/Web/API/GPUTexture/createView
Title: GPUTexture.createView()


URL: /en-US/docs/Web/API/GPUTexture/format
Title: GPUTexture.format


URL: /en-US/docs/Glossary/Bitwise_flags
Title: Bitwise flags

(comment last updated: 2023-04-05 12:27:06)

Copy link
Contributor

@dawei-wang dawei-wang left a comment

Choose a reason for hiding this comment

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

A few typos

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.

Looking good!

@github-actions github-actions bot added the Content:Glossary Glossary entries label Mar 13, 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.

Awesome, thanks for the updates! This change overall LGTM now! (Can't actually approve because it's a draft)

@chrisdavidmills
Copy link
Contributor Author

Awesome, thanks for the updates! This change overall LGTM now! (Can't actually approve because it's a draft)

Great, thanks!

@chrisdavidmills chrisdavidmills marked this pull request as ready for review April 3, 2023 08:54
@chrisdavidmills chrisdavidmills requested a review from a team as a code owner April 3, 2023 08:54
@chrisdavidmills chrisdavidmills force-pushed the webgpu-docs-2-resources branch from de4f813 to 1c210af Compare April 5, 2023 11:54
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 this per slack discussion 👍

@Elchi3 Elchi3 merged commit 3996d67 into mdn:main Apr 5, 2023
@chrisdavidmills chrisdavidmills deleted the webgpu-docs-2-resources branch April 5, 2023 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Glossary Glossary entries Content:WebAPI Web API docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants