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.

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

mtl: Map more vertex formats #1985

Merged
merged 1 commit into from
May 4, 2018
Merged

Conversation

grovesNL
Copy link
Contributor

@grovesNL grovesNL commented May 4, 2018

Depends on gfx-rs/metal-rs#41 to be merged/published.

Not sure what to do with Int1010102Normalized and UInt1010102Normalized because the byte ordering for alpha appears to differ from Vulkan, so I'm leaving them out for now.

PR checklist:

  • make succeeds (on *nix)
  • make reftests succeeds
  • tested examples with the following backends: Metal

@kvark
Copy link
Member

kvark commented May 4, 2018

I published https://crates.io/crates/metal-rs/0.8.3 and added you to the owners of metal-rs.
The format issues are added to #1354
bors r+

bors bot added a commit that referenced this pull request May 4, 2018
1985: mtl: Map more vertex formats r=kvark a=grovesNL

Depends on gfx-rs/metal-rs#41 to be merged/published.

Not sure what to do with `Int1010102Normalized` and `UInt1010102Normalized` because the byte ordering for alpha appears to differ from Vulkan, so I'm leaving them out for now.

PR checklist:
- [X] `make` succeeds (on *nix)
- [X] `make reftests` succeeds
- [X] tested examples with the following backends: Metal


1987: dx12: Implement buffer views r=kvark a=msiglreith

Untested so far, probably requires small changes to portability due to API breakage.


Co-authored-by: Joshua Groves <josh@joshgroves.com>
Co-authored-by: msiglreith <m.siglreith@gmail.com>
@bors
Copy link
Contributor

bors bot commented May 4, 2018

Build failed (retrying...)

bors bot added a commit that referenced this pull request May 4, 2018
1985: mtl: Map more vertex formats r=kvark a=grovesNL

Depends on gfx-rs/metal-rs#41 to be merged/published.

Not sure what to do with `Int1010102Normalized` and `UInt1010102Normalized` because the byte ordering for alpha appears to differ from Vulkan, so I'm leaving them out for now.

PR checklist:
- [X] `make` succeeds (on *nix)
- [X] `make reftests` succeeds
- [X] tested examples with the following backends: Metal


Co-authored-by: Joshua Groves <josh@joshgroves.com>
@bors
Copy link
Contributor

bors bot commented May 4, 2018

@bors bors bot merged commit b00d36d into gfx-rs:master May 4, 2018
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.

2 participants