Skip to content

Meeting 2021-02-18 #428

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

Closed
XAMPPRocky opened this issue Feb 15, 2021 · 1 comment
Closed

Meeting 2021-02-18 #428

XAMPPRocky opened this issue Feb 15, 2021 · 1 comment
Labels
t: meeting Meeting agendas & minutes

Comments

@XAMPPRocky
Copy link
Member

XAMPPRocky commented Feb 15, 2021

Where & When

Agenda

Feel free to propose additional items in this thread.

@XAMPPRocky XAMPPRocky added the t: meeting Meeting agendas & minutes label Feb 15, 2021
@XAMPPRocky
Copy link
Member Author

Project Updates:

@Jasper-Bekkers: Byte address buffers

  • I haven't checked the SPIR-V
  • the parameter loading is the offset in bytes aligned to word.
  • The traverse engine has switched to fully bindless.
  • Glad I got my PR merged.

@eddyb: Now I have to make it work with my stuff 😄

  • It's not too bad, I just need to update some code in tests.

@Jasper-Bekkers: One things that is a non uniform resource asset. Where instead of a uniform lookup and index, you can get a nonuniform one.
@eddyb: I don't think that interacts with storage class inference.
@Jasper-Bekkers: Probably not.
@eddyb: There are some todos, and it's to easy to trigger by disabling DCE.
@khyperia: Those shouldn't be blockers though.
@eddyb: It just makes it easier, I think I have a path forward.

@khyperia: Mostly working on Ark.
@VZout: Same
@Jasper-Bekkers: Making pr
@XAMPPRocky: Added documentation around inline asm, added OpImageRead/OpImageWrite and StorageImage type.

MCPs:

#429

@XAMPPRocky: I'd for us to start using completest for tests, it has a variety of features, and importantly it should enable our tests to be able to run parallel rather than serial.
@khyperia: Seems good.
@eddyb: Might need to figure out codegen tests, as that uses a LLVM tool.
@XAMPPRocky: Supporting other codgen might be nice a addition to compiletest if it doesn't work out of the box.

Other

@Hentropy: Regarding #423

  • So this would provide significant compile-time benefits, but there needs to be logic copied from spirv-builder into the compile_shader function? Does that seem okay?

@khyperia: Yep

@Hentropy: should the ash runner be moved to a test runner from an example runner?
@eddyb: We could have a common crate for applying events from the Window, and for headless we can just pass PushConstant with some values.

  • Doesn't need to be complicated.
  • You can keep all the ash rendering code, but that's it. It does get tricky with UniformConstants

@Jasper-Bekkers: As long as the ash code is run I don't see a problem.

@XAMPPRocky: Should we have a nominated label for tagging anything you want to discuss at the next meeting?
@khyperia: Seems good.

@Hentropy: Has there been much though into how to share CPU and GPU code.
@VZout: We've been working on some different approaches internally in Ark.
@XAMPPRocky: Of course, if other people have ideas, please share the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t: meeting Meeting agendas & minutes
Projects
None yet
Development

No branches or pull requests

1 participant