torch-mlir icon indicating copy to clipboard operation
torch-mlir copied to clipboard

emitted aten.norm.ScalarOpt_dim

Open albertdmath opened this issue 3 years ago • 10 comments

albertdmath avatar Jun 16 '22 16:06 albertdmath

Can you rebase your branch to resolve the conflicts so that the CI can run?

ramiro050 avatar Jun 29 '22 17:06 ramiro050

Can you rebase your branch to resolve the conflicts so that the CI can run?

ramiro050 avatar Jul 20 '22 18:07 ramiro050

Can you add the div.int op in a separate PR? This makes the PRs more self-contained and easier to review

ramiro050 avatar Aug 08 '22 17:08 ramiro050

Hey @albertdmath, you can also run the lit tests locally by doing cmake --build build --target check-torch-mlir-all.

ramiro050 avatar Aug 12 '22 17:08 ramiro050

The error I've been getting is that when I use floats the error calls for ints and when I use ints it calls for floats.

albertdmath avatar Aug 12 '22 17:08 albertdmath

Just to confirm, the cast would happen with AtenToDtypeOp?

albertdmath avatar Aug 15 '22 19:08 albertdmath

I'm also assuming this new approach would mean reverting the changes I made to TorchOps.cpp.

albertdmath avatar Aug 15 '22 20:08 albertdmath

Just to confirm, the cast would happen with AtenToDtypeOp?

Can you create the lowering for AtenDivInt in a separate PR, since it is not really related to the goal of this PR.

AtenToDtypeOp takes a tensor as an input, so that would not work with your op. You should be able to use AtenFloatScalar to cast the scalars to float

ramiro050 avatar Aug 15 '22 21:08 ramiro050

Alright, I've created a separate PR for aten.div.int, please let me know if it looks good. Since this op depends on aten.div.int, I first want to get aten.div.int merged in, then take out the work for aten.div.int from this branch and get this merged in.

albertdmath avatar Aug 17 '22 13:08 albertdmath

Any progress with this PR?

Vremold avatar Oct 10 '22 15:10 Vremold