jax icon indicating copy to clipboard operation
jax copied to clipboard

Custom gpu ops partitioning

Open keshavb96 opened this issue 11 months ago • 3 comments

PR to update tutorial to demonstrate the use of custom_partitioning instead of xmap when writing a custom primitive in JAX

keshavb96 avatar Mar 11 '24 20:03 keshavb96

General comment: should we perhaps choose a simpler op/ops for this? RMSNorm needs a lot of CUDA code which is not directly relevant here.

People don't need to read the CUDA code here. If we took something simpler, we won't be able to show all features, like multi-output.

If we find something simpler that still show all features, that would be great. But I don't have an idea. But this should be a follow up PR I think.

nouiz avatar Mar 15 '24 14:03 nouiz

General comment: should we perhaps choose a simpler op/ops for this? RMSNorm needs a lot of CUDA code which is not directly relevant here.

People don't need to read the CUDA code here. If we took something simpler, we won't be able to show all features, like multi-output.

If we find something simpler that still show all features, that would be great. But I don't have an idea. But this should be a follow up PR I think.

@jaro-sevcik had similar perception I believe ... any ideas here?

mjsML avatar Mar 15 '24 14:03 mjsML

If we took something simpler, we won't be able to show all features, like multi-output.

What are some of the other features we want to demonstrate?

If we find something simpler that still show all features, that would be great. But I don't have an idea. But this should be a follow up PR I think.

Yeah, I agree this could be done in a follow up.

superbobry avatar Mar 18 '24 09:03 superbobry

@keshavb96 is the PR ready for another review?

nouiz avatar Mar 28 '24 14:03 nouiz

Yes, I think it should be good to go

keshavb96 avatar Mar 28 '24 18:03 keshavb96

Can you squash the commits, please? I suspect that could be the reason for the import/copybara check failing.

superbobry avatar Apr 02 '24 13:04 superbobry

@superbobry It still seems like copybara is failing despite squashing all the commits

keshavb96 avatar Apr 05 '24 18:04 keshavb96

It looks like Copybara is unhappy that some files are missing the Apache license header. Can you make sure all files have one, please?

superbobry avatar Apr 07 '24 17:04 superbobry

Can you squash the commits, please?

superbobry avatar Apr 08 '24 08:04 superbobry

Done!

keshavb96 avatar Apr 08 '24 11:04 keshavb96