Why python operator defines does not use these in test/common/src? about triton HOT 5 CLOSED

Oh, the triton-ops branch is not merged in yet. I will remove the C++ tests when I merge it into master and move all the tests/benchmarks to pytest :)

from triton.

Oh, the triton-ops branch is not merged in yet. I will remove the C++ tests when I merge it into master and move all the tests/benchmarks to pytest :)

Sometimes, We need to call operator in c++. So I suggest to keep the operator defines in c file. and parse them in python.

from triton.

I think you are right. My goal eventually is to have a .c file for each triton op defined in Python. I think right now there is an issue that prevents from having multiple kernels in the same .c file, but I will fix that.

PS: I have created a Slack workspace for Triton by the way. Feel free to e-mail me at [email protected] if you want an invite.

from triton.

I write a method, you can reference it.

from triton.

Thanks for your comment and suggestion. Everything is now more modular and includes .c file for the kernels in the python tree. It is also now possible to select by name which kernel to read from the source file as shown here https://github.com/ptillet/triton/blob/master/python/triton/ops/blocksparse/softmax.py . This is done in the C++ binding here https://github.com/ptillet/triton/blob/master/python/src/bindings.cc#L119 .

Closing the issue for now, but feel free to reopen if you have any related comment

from triton.