They got to him

  • ooterness@lemmy.world
    link
    fedilink
    English
    arrow-up
    144
    arrow-down
    27
    ·
    1 day ago

    LLMs make slop code faster than humans can meaningfully review. If you make LLMs the only review layer, then slop gets through because they have similar biases. Stop making slop.

    • NuanceDemon@lemmy.dbzer0.com
      link
      fedilink
      English
      arrow-up
      9
      arrow-down
      4
      ·
      22 hours ago

      You had to lean on that assumption that there’s a massive amount being produced to make this point though.

      The actual operating point is that the slop is made faster than humans can WRITE it.

      Reviewing 100 lines of code thats 90% there and took 3 minutes to produce is faster than writing it from scratch, there’s no getting around that.

      For me the issue with it is that all (hyperbole) work is about to become reading instead of creating, dull.

      • sem@lemmy.blahaj.zone
        link
        fedilink
        English
        arrow-up
        5
        ·
        18 hours ago

        You might save time in writing the code, but you lose time from the code doing things that aren’t in your mental model of the program

    • auzy1@lemmy.world
      link
      fedilink
      English
      arrow-up
      57
      arrow-down
      33
      ·
      1 day ago

      Cool assumption. But pretty sure it’s not how things work at all but sounds great…

      What actually happens is that you have a long list of multiple maintainers reviewing , and multiple branches for testing, and multiple layers of review too.

      https://www.kernel.org/doc/html/next/process/maintainers.html . See that long list of maintainers for a start.

      The actual policy is that everything ends up getting reviewed by a human. If I post a patch, it doesn’t go straight to master

      If it’s not being properly tested and maintained, it gets dropped irrespective of what it is (remember devfs?)

      Also, I’m not sure about the kernel specifically, but on projects I submitted code to, they often wanted complex patches broken down if possible.

      • darkangelazuarl@lemmy.world
        link
        fedilink
        English
        arrow-up
        49
        ·
        1 day ago

        Yes this means the review process is working. What is important to track is if LLM code causes a disproportionately large number of reviews or reviews that take longer because whoever generated the code doesn’t actually know what it is doing. Hopefully not the case here but it does happen a lot with LLM coding.

        • datavoid@sh.itjust.works
          link
          fedilink
          English
          arrow-up
          13
          arrow-down
          23
          ·
          1 day ago

          Hopefully not the case here but it does happen a lot with LLM coding.

          Linus secretly understands nothing about the Linux kernel

      • nilloc@discuss.tchncs.de
        link
        fedilink
        English
        arrow-up
        15
        arrow-down
        11
        ·
        1 day ago

        XZ Utils made it past all those checks with malicious code from a human.

        If they’d been more careful with their tarball commits it probably would have made it into the final and commercial distros.

        It made into Homebrew for instance.