this post was submitted on 22 Nov 2025
143 points (98.6% liked)

Programming

24265 readers
886 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev

founded 2 years ago
MODERATORS
snowe@programming.dev
Ategon@programming.dev
MaungaHikoi@lemmy.nz
UlrikHD@programming.dev
143
OpenAI Demo'd Fixing Issue #2472 Live. It's Still Open Months Later. (blog.tymscar.com)
submitted 1 month ago by Kissaki@programming.dev to c/programming@programming.dev
15 comments fedilink hide all child comments
 

During OpenAI’s GPT-5 launch event, they demoed the model’s ability to fix real bugs in production code. Live on stage. In their own repository. The kind of demo that makes CTOs reach for their credit cards and engineers nervously update their resumes. There’s just one small problem: the fix they promised to merge “right after the show” is still sitting there, unmerged, three and a half months later.

top 15 comments
sorted by: hot top controversial new old
[–] Kissaki@programming.dev 33 points 1 month ago (4 children)

The issue, presumably the PR (linked at the top of the issue because of reference).

Look at the code change. It gets inputs and loops over them and seems to do an in-place fixup. But the code indent is wrong, and it even changed the function definition of the unrelated next function. In Python, the indent-logic-significance language.

I assume they briefly showed the code on stage. Even then it should have been obvious to any developer. py file, messy indent, changes unrelated function.

Please correct me if this is the wrong PR.

[–] thebeardedpotato@lemmy.world 20 points 1 month ago (1 children)

Based on what they showed in the demo, that’s not the PR. gpt made more changes than what’s in that PR and also modified a different transform function. My guess is they never actually pushed the actual commit from the demo or made a PR.

[–] Kissaki@programming.dev 1 points 1 month ago

I see, thank you for the clarification. I was quite confused because it seemed to be missing, this one didn’t quite seem correct. If they never even pushed it as a MR then that makes more sense. Then the whole “hasn’t been merged yet” is missing that it hasn’t even been created.

[–] Yoddel_Hickory@piefed.ca 15 points 1 month ago* (last edited 1 month ago) (1 children)

Actually the function definition is unchanged. The line that was "added" at the bottom was also "removed" at the top. This is just the Git diff generator being confused, which won't come as a surprise to anyone that has ever used it.

The indendentation really is messed-up though.

[–] Kissaki@programming.dev 1 points 1 month ago (1 children)

An indentation change is a definition code change. And as I pointed out, it's a py file, and Python is an indent-significant language.

[–] Yoddel_Hickory@piefed.ca 1 points 1 month ago* (last edited 1 month ago) (1 children)

Of course, but you said:

But the code indent is wrong, and it even changed the function definition of the unrelated next function.

It is weird to split the two in your sentence, as only the indentation of the next function definition was changed, not the definition itself.

You can just take the L and say you didn't see that the function definition that was "added" was just "removed" at the top. It is an easy mistake to make, I know I've done it many times.

[–] Kissaki@programming.dev 1 points 1 month ago

You can just take the L and say you didn’t see that the function definition that was “added” was just “removed” at the top.

That's not what happened though.

Changing the indent of the def changes the definition. That's my whole argument.

I don't get why you say "of course", agreeing with my point, but then "it was only the indentation that was changed".

[–] Zikeji@programming.dev 13 points 1 month ago (1 children)

Does not appear to be the correct MR. Comments on the issue allude to "they never pushed it" so sounds like there never was an MR. Watching the announcement where they demo'd it, it wrote much more than is in that MR. Not to defend OpenAI, I hate vibe coded solutions that add so many useless comments.

Write. Readable. Code.

[–] Kissaki@programming.dev 1 points 1 month ago

I see, thank you for the clarification. I was quite confused because it seemed to be missing, this one didn't quite seem correct. If they never even pushed it as a MR then that makes more sense. Then the whole "hasn't been merged yet" is missing that it hasn't even been created.

[–] A_norny_mousse@feddit.org 1 points 1 month ago

The issue

What's with the comments? "Hi mom"

the PR

So "LuminaX-alt" is an AI?

[–] kureta@lemmy.ml 29 points 1 month ago (3 children)

Imagine if a FAANG company pulled this move - demo a feature on stage, promise to ship it, then just… don’t.

I'm still waiting for Google assistant to make me a restaurant reservation. It has been 7 years.

[–] FizzyOrange@programming.dev 5 points 1 month ago (1 children)

I suspect in the real world it's frustrating enough for restaurants that it wouldn't have worked out.

You're pretty much tricking restaurant workers into one of those awful voice-based phone trees.

Plus there are so many things that can actually happen when you try to book a table on the phone - they don't have exactly what you want but can offer you this time instead... they only have outside seating available... etc. etc.

Plus, just having a proper online booking form is clearly a better option and not totally uncommon these days.

[–] 123@programming.dev 2 points 1 month ago* (last edited 1 month ago)

They saw restaurant workers as just a cog in their machine that would put up with an error prone voice assistant. Hard to get that from minimum wage workers.

On my area some restaurants have just stopped answering the phone that you could previously use to place orders. I'd do the same (stop caring) if I could barely afford life even with a bunch if roommates.

[–] brian@programming.dev 0 points 1 month ago

I'm pretty sure it can, it's just been moved to Google maps instead