Sitelet https://github.com/gpuweb/gpuweb/pull/680
Skip to content

Add #buffer-uploads about mapping in modern graphics APIs and copyDataToBuffer example. - #680

Closed
kdashg wants to merge 1 commit into
gpuweb:masterfrom
kdashg:spec-buffer-upload-rec
Closed

kdashg wants to merge 1 commit into
gpuweb:masterfrom
kdashg:spec-buffer-upload-rec

Conversation

@kdashg

@kdashg kdashg commented Apr 1, 2020 •

Copy link
Copy Markdown
Contributor

I keep wanting to refer back to this recommendation for easy buffer uploads, so I think it makes sense to propose some non-normative spec text about it.

This applies to the state of the spec today, and may or may not change based on ongoing work on buffer upload entrypoints.


Preview | Diff

@kvark kvark 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.

I agree this material could be useful. It's not obvious to me that it needs to be in the specification though, perhaps a tutorial or some kind of "best practices" document would make more sense?

I also don't think we want to put this in while the heated discussions about uploads are still on-going.
The group mostly agrees that, given the uploading is one of the frequently needed tasks, it makes sense for the API to provide a simple mechanism for it, even if there is a more complex approach that could be more efficient. And once we decide what this mechanism is (writeToXxx, inlineUploadBuffer, or something else), this paragraph would have to be scraped anyway.

@kainino0x

Copy link
Copy Markdown
Contributor

I think this should stay unlanded until the writeToX discussion, so removed for-meeting. Does that make sense?

@Kangz

Kangz commented May 20, 2020

Copy link
Copy Markdown
Contributor

Closing now that #708 is landed.

@Kangz Kangz closed this May 20, 2020
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