Skip to content

[editorial] Clearly distinguish texel copy footprints vs memory costs#3922

Merged
kainino0x merged 2 commits intogpuweb:mainfrom
kainino0x:stencil8-size
Mar 7, 2023
Merged

[editorial] Clearly distinguish texel copy footprints vs memory costs#3922
kainino0x merged 2 commits intogpuweb:mainfrom
kainino0x:stencil8-size

Conversation

@kainino0x
Copy link
Copy Markdown
Contributor

And reorganize the texture format tables a little bit.

Fixes #3768

And reorganize the texture format tables a little bit.

Fixes 3768
@kainino0x kainino0x added the copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) label Mar 4, 2023
@kainino0x kainino0x added this to the V1.0 milestone Mar 4, 2023
@kainino0x kainino0x requested a review from toji March 4, 2023 03:28
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 4, 2023

Previews, as seen when this build job started (4c42adf):
WebGPU webgpu.idl | Explainer | Correspondence Reference
WGSL grammar.js | wgsl.lalr.txt

@teoxoy
Copy link
Copy Markdown
Member

teoxoy commented Mar 4, 2023

Nicely done!

Copy link
Copy Markdown
Member

@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 with one nit

<td>2
<td>&ndash; <!-- no render target -->
<td>&ndash; <!-- no render target -->
<td colspan=1>2
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any reason for the colspan=1 here and elsewhere?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The colspan attribute triggers a CSS rule we defined to add dotted lines on either side of the cell. This makes the dotted lines consistent through the columns. Though I'm considering just adding vertical lines in all tables all the time because this is getting kind of silly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarification of stencil8 texel block size

3 participants