Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Align the C header with Dawn #6

Closed
kvark opened this issue Jan 23, 2019 · 6 comments
Closed

Align the C header with Dawn #6

kvark opened this issue Jan 23, 2019 · 6 comments

Comments

@kvark
Copy link
Member

kvark commented Jan 23, 2019 •
edited
Loading

https://github.com/webgpu-native/webgpu-headers
@grovesNL
Copy link
Collaborator

Probably need to wait until the IDL receives the next few updates before we spend much time on this :)

@DagAgren
Copy link

Is there any progress on this? I'd like to try playing around with using wgpu-native a bit, and it would be nice if the API was getting close to wherever it is going.

@kvark
Copy link
Member Author

kvark commented Jul 14, 2019 via email

@kvark kvark transferred this issue from gfx-rs/wgpu Apr 21, 2020
@kvark kvark mentioned this issue Apr 21, 2020
Closed
@kvark kvark mentioned this issue May 2, 2020
Closed
@kvark kvark pinned this issue May 3, 2020
@paulkernfeld
Copy link

I wasn't able to find the dawn.h file linked at the top of the issue. I think it's now something within this folder but I'm not sure what.

@kvark
Copy link
Member Author

kvark commented May 12, 2020

@paulkernfeld I updated the subject with the actual path.

@kvark kvark mentioned this issue May 31, 2020
Closed
@kvark kvark mentioned this issue Oct 21, 2020
Closed
@kvark kvark mentioned this issue Dec 2, 2020
Closed
@kvark kvark mentioned this issue Feb 20, 2021
Merged
bors bot added a commit that referenced this issue Feb 21, 2021
@bors
Merge #75
fb0876a 
75: Migrate pipeline creation to align with webgpu-headers r=kvark a=Sineaggi

Edit(@kvark): solid step towards #6 

First effort trying to better align wgpu-native with webgpu-headers. Snake case was replaced with camel case, and the descriptor was rewritten to be closer to the structs that are defined in webgpu-headers.

Only the single function `wgpuDeviceCreateRenderPipeline` was changed in this pr to gauge interest. If merged, we'd be leaving wgpu-native in an inconsistent state.

Co-authored-by: Clayton Walker <cwalker@sofi.org>
@kvark kvark mentioned this issue Feb 22, 2021
Closed
grovesNL added a commit to grovesNL/wgpu-native that referenced this issue Mar 23, 2021
@bors @grovesNL
Merge gfx-rs#10
fc3b6fc 
10: Generate C header r=kvark a=grovesNL

Fixes gfx-rs#6

~~WIP: there are a few issues I've encountered already for the few functions we expose.~~

Co-authored-by: grovesNL <josh@joshgroves.com>
Co-authored-by: Joshua Groves <josh@joshgroves.com>
@20kdc 20kdc mentioned this issue Mar 21, 2022
Open
@cwfitzgerald
Copy link
Member

We now gen from there, so this is good

@cwfitzgerald cwfitzgerald unpinned this issue Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kvark @paulkernfeld @grovesNL @cwfitzgerald @DagAgren