Isuru Fernando
Isuru Fernando
I changed to `_unsafe_masked_index_put_accumulate`. Do you think we should also remove the explicit `mask`?
Added 3 PRs to the stack here to show the use of the operator.
Any idea how to fix the compilation failures?
This is ready for another review.
@pytorchbot merge
> inductor/test_torchinductor_opinfo.py::TestInductorOpInfoCPU::test_comprehensive__unsafe_masked_index_cpu_bool This test failure is real. I can reproduce locally on linux (It's strange that this error came up only on macos run) The failure seems to come from...
> Is it an easy fix? If so, can you prepend a PR with the fix? Not sure. I opened https://github.com/pytorch/pytorch/pull/126406, but it's most probably not the correct solution.
[BUG] - crash on c++ exception when importing 2 pybind submodules built from different MSVC versions
@pwuertz, TensorRT was using /MT (which is a strange choice IMO) whereas all the others are using `/MD`. I've sent https://github.com/pybind/pybind11/pull/4953 to fix this.
I think it's better to implement this as a conda package instead of as yet another configuration option. For eg: see https://github.com/conda-forge/conda-ecosystem-user-package-isolation-feedstock
conda-build is supposed to use `miniconda3\envs\b1\conda-bld\jq-suite_1695720509023\_h_env\Library\usr\bin\bash.exe`. Is it not?