Comments by malfet

All comments ranked by humor rating

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong?"

malfet13 days agopytorch/pytorch

Deleted code is tested code!

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Let's test in pro-o-o-o-o-d!"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Works? Works?? Works!"

malfet1 day agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod!"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Deleted code is tested code"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Lint is green, XPU is red"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "We can only test it in prod"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong here"

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "ci failures unrelated"

@bobrenjc93 why do you think 100+ failures are all unrelated to your PR?

malfet2 days agopytorch/pytorch

@pytorchbot merge -f "¯\(ツ)/¯ ¯\(ツ)/¯ ¯\(ツ)/¯ "

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod, we'll not know the answer until 7 hours later"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -i

What can possible go wrong?

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Sparta AaA aaaaa...."

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "let's test in prod"

malfet3 months agopytorch/pytorch

Deleted code is tested code :)

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Bibbidi! Bobbidi... boo!"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "Roses are red, violets are blue, MPS is green, so I merge you"

malfet6 months agopytorch/pytorch

need to remove pytorch submodule

I wonder if

git submodule update --init --recursive
will delve into infinite recursion with that one :P

malfet23 days agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod, last run only had missing lint"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Roses are red, violets are blue, I need to come up with better merge -f comments"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "I don't know what it's unhappy about, but hopefully this is OK"

malfet30 days agopytorch/pytorch

@pytorchbot merge -f "I'm pretty sure I know how to fix lint"

malfetabout 1 month agopytorch/pytorch

Deleted code is tested code

malfetabout 1 month agopytorch/pytorch

I love the smell of gdb in the morning... Copy-paste mistake :P

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Spartaaaaaa!"

malfet3 months agopytorch/pytorch

Grrabbing this issue I think I know how to handle this

If you are grabbing it for yourself, why I got assigned to it ;P

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "What could possible go wrong?"

malfet3 months agopytorch/pytorch

Supporting gcc1 was a special request from Dr Emmet Brown who is looking forward to build PyTorch from source back in 1987

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "I wonder what can go wrong"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "I'll be surprised if it can regress ROCm"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "2nd time is the charm, sorry for causing the land race"

malfet27 days agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "Sparta!!! (i.e. let's test in trunk)"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "This should not work!"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "2nd time is the charm?"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Abracadabra"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Deleted code is tested code"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "Lint is green, XPU is red, but for rest of CI it's a no-op"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "Am I too impatient?"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "It looks green"

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "Let's risk it"

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "Lint is sad now"

malfet6 months agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod"

malfet6 months agopytorch/pytorch

lgtm

It's nice to see you approve of your own PRs :P

malfet2 months agopytorch/pytorch

Why this is needed?

rm -rf
on the previous line should be enough?

malfet3 days agopytorch/pytorch

@pytorchbot merge -f "This seems fine?"

malfet3 days agopytorch/pytorch

@pytorchbot merge -f "Let's see what will happen"

malfet4 days agopytorch/test-infra

sys.exit(-1)
seems to be in order here

malfet4 days agopytorch/pytorch

@pytorchbot merge -f "This is probably fine"

malfet10 days agopytorch/pytorch

@pytorchbot merge -f "Lint + aarch64 tests are green, and trunk, well.."

malfet16 days agopytorch/pytorch

@pytorchbot revert -m "I want to revert this change as I'm 90+% certain it somehow broke testing" -c weird

malfet26 days agopytorch/pytorch

@pytorchbot merge -f "This looks fine"

malfet30 days agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "Lint is green"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "I hope I know what I'm doing"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Hope it will not break anything, but I really don't know"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "How bad could it be?"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "It's probably fine"

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "Embarrassed by further typos"

malfet6 months agopytorch/pytorch

Well,

torch.tensor(-1).rsqrt()
should clearly return Nan, not sure what went wrong afterwards (on M1 it returns expected values)

malfet7 months agopytorch/pytorch

Have you checked it's not due to the broke trunk?

malfet2 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong I wonder?"

malfet6 months agopytorch/pytorch

@pytorchbot merge -f "This PR was green in the past"

malfet1 day agopytorch/pytorch

@pytorchbot merge -f "Trunk is red, Tests are green, Merging this PR— Best I've seen!"

malfet5 days agopytorch/pytorch

@pytorchbot revert -m "I'm blind" -c ignoredsignal

malfet23 days agopytorch/pytorch

@pytorchbot merge -f "Let's test those two in prod"

malfet23 days agopytorch/pytorch

@pytorchbot merge -f "I'm impatient"

malfet25 days agopytorch/pytorch

@pytorchbot merge -f "All is green, and trunk will not help here"

malfet27 days agopytorch-labs/helion

@pytorchbot merge -f "Roses are red, violets are green"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod, as TD might have skipped a lot of relevant tests"

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "All is green, not sure what it's waiting for"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Pull is green, what more to wish for?"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Looks like lint is green, and rest is a mistery"

malfetabout 2 months agopytorch/pytorch

Sure, may be it'll work, may be it would not...

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Keeping my fingers crossed"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "XPU builds seems appeased"

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "Where are my pull request jobs?"

malfetabout 2 months agopytorch/pytorch

@pytorchmergebot merge -f "mergebot was updated, will it work now?"

malfet2 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong I wonder?"

malfet2 months agopytorch/pytorch

Sure, if lint is green...

malfet3 months agopytorch/pytorch

No C-style enums in 2025 please

enum class Kind {
malfet3 months agopytorch/pytorch

Looks definatly good to me (type intended :) )

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "MacOS builds are green, rest of the failures are irrelevant"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "2nd time is the charm"

malfet3 months agopytorch/pytorch

@pytorchbot revert -m "I'm unsure why this PR got merged, as it doesn't have a valid review" -c weird

malfet3 months agopytorch/pytorch
::metal::precise::sqrt(::metal::fabs(x));
malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Let's test in prod! Spartaaa!"

malfet3 months agopytorch/pytorch

This sounds like an oxymoron: why CPU builds needs CUDA (and what this config is for to begin with)

malfet4 months agopytorch/pytorch

@pytorchbot revert -m "Broke lint?, I guess land race with rufff update" -c nosignall

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "Lint is green, let's test in prod"

malfet4 months agotriton-lang/triton

All dates are tentative -> all future release dates are netative.

malfet5 months agopytorch/pytorch

@pytorchbot revert -m "Want to refine it a bit" -c weird

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "It's just a rebase"

malfet5 months agopytorch/pytorch

@pytorchbot merge -f "How badly can I mess up if signal were green before?"

malfet5 months agopytorch/pytorch

@Skylion007 , well, according to VC++2019 it's

not true
(pun intended) :)