Skip to content

Add the buffer mapping explainer - #1600

Merged
kvark merged 4 commits into
gpuweb:mainfrom
Kangz:mapping_explainer
Apr 8, 2021
Merged

kvark merged 4 commits into
gpuweb:mainfrom
Kangz:mapping_explainer

Conversation

@Kangz

@Kangz Kangz commented Apr 6, 2021

Copy link
Copy Markdown
Contributor

Includes extensive copyediting by @kainino0x

@Kangz
Kangz requested review from kainino0x and kvark April 6, 2021 16:25

@kainino0x kainino0x left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assuming no major edits since I copyedited. LGTM including formatting stuff.

Comment thread explainer/index.bs Outdated
Comment thread explainer/index.bs Outdated
Comment thread explainer/index.bs Outdated
Comment thread explainer/index.bs Outdated
@Kangz

Kangz commented Apr 8, 2021

Copy link
Copy Markdown
Contributor Author

Addressed all comments, please take a look or merge!

@kvark
kvark merged commit 49ece0d into gpuweb:main Apr 8, 2021
@kainino0x kainino0x mentioned this pull request Apr 8, 2021
16 tasks
@Kangz
Kangz deleted the mapping_explainer branch March 25, 2022 14:21
ben-clayton pushed a commit to ben-clayton/gpuweb that referenced this pull request Sep 6, 2022
…attachment (gpuweb#1600)

This PR adds test cases that check mipsLevelCount: 0 and arrayLayerCount: 0
to the render_pass,resolve:resolve_attachment test. A validation error should
be generated if resolveTargetViewMipCount or resolveTargetViewArrayLayerCount
is 0.

Issue: 1599
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.

3 participants