Comments by malfet

All comments ranked by humor rating

malfet3 days agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfetabout 2 months agopytorch/pytorch

Deleted code is tested code!

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfetabout 2 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet7 months agopytorch/pytorch

@pytorchbot merge -f "ci failures unrelated"

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

malfetabout 2 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

@pytorchbot merge -i

What can possible go wrong?

malfet3 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

Deleted code is tested code :)

malfet4 months agopytorch/pytorch

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

malfet5 months agopytorch/pytorch

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

malfet8 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

malfet2 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "What can go wrong"

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet7 days agopytorch/pytorch

IMO one simple solution is to disable the linter and sent it back to the author...

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

Deleted code is tested code

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Spartaaaaaa!"

malfet4 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

malfet4 months agopytorch/pytorch

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

malfet5 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

malfet5 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet2 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfet4 months agopytorch/pytorch

@pytorchbot merge -f "Abracadabra"

malfet4 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

@pytorchbot merge -f "It looks green"

malfet7 months agopytorch/pytorch

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

malfet7 months agopytorch/pytorch

@pytorchbot merge -f "Lint is sad now"

malfet8 months agopytorch/pytorch

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

malfet8 months agopytorch/pytorch

lgtm

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

malfet4 months agopytorch/pytorch

Why this is needed?

rm -rf
on the previous line should be enough?

malfet9 days agopytorch/pytorch

That's the plan, but I hugely suspect I'll have to leave the skip for MacOS-13, where 4GB tensors are big taboo

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "This seems fine?"

malfetabout 2 months agopytorch/pytorch

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

malfetabout 2 months agopytorch/test-infra

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

malfetabout 2 months agopytorch/pytorch

@pytorchbot merge -f "This is probably fine"

malfetabout 2 months agopytorch/pytorch

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

malfet2 months agopytorch/pytorch

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

malfet2 months agopytorch/pytorch

@pytorchbot merge -f "This looks fine"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Lint is green"

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "This is fine"

malfet3 months agopytorch/pytorch

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

malfet5 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

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

malfet6 months agopytorch/pytorch

@pytorchbot merge -f "Embarrassed by further typos"

malfet7 months agopytorch/pytorch

Well,

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

malfet8 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet7 months agopytorch/pytorch

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

malfet4 days agopytorch/pytorch

Why not add

3.14t
there as well?

malfet25 days agopytorch/pytorch

Grabbing to myself to identify offending PR and add deprecation function

malfetabout 1 month agomalfet/deleteme

@pytorchbot revert -m "I still want it" -c weird

malfetabout 1 month agopytorch/pytorch

To be frank, Python-3.14 is not out yet, but yeah, we should be looking into it

malfetabout 1 month agopytorch/pytorch

@pytorchbot merge -f "It was green before the rebase"

malfetabout 2 months agopytorch/pytorch

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

malfetabout 2 months agopytorch/pytorch

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

malfet2 months agopytorch/pytorch

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

malfet2 months agopytorch/pytorch

@pytorchbot merge -f "I'm impatient"

malfet2 months agopytorch/pytorch

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

malfet2 months agopytorch-labs/helion

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "Keeping my fingers crossed"

malfet3 months agopytorch/pytorch

@pytorchbot merge -f "XPU builds seems appeased"

malfet3 months agopytorch/pytorch

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

malfet3 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

Sure, if lint is green...

malfet4 months agopytorch/pytorch

No C-style enums in 2025 please

enum class Kind {
malfet4 months agopytorch/pytorch

Looks definatly good to me (type intended :) )

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

malfet4 months agopytorch/pytorch

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

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

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