• RampantParanoia2365@lemmy.world
    link
    fedilink
    arrow-up
    6
    ·
    22 hours ago

    I think my least favorite part of Lemmy is all the posted screenshots of tweets. It just took me 3 tries to get to these comments.

    What benefit do tweet screenshots serve?

    • poke@sh.itjust.works
      link
      fedilink
      arrow-up
      12
      ·
      22 hours ago

      I know what context they are looking at and I don’t need to make a twitter account to view the relevant replies.

  • CXORA@aussie.zone
    link
    fedilink
    English
    arrow-up
    310
    ·
    2 days ago

    People who share the size of a codechange as a mark of how effective ai coding agents are truly missing the point of code changes.

    • Redkey
      link
      fedilink
      arrow-up
      5
      ·
      1 day ago

      You reminded me of a story I recently read, where the author highlighted just how much awesome programming someone had done by describing how their hands were cramping up.

      It’s like estimating how well an artist paints by looking at how much paint is on their clothes, or judging how good a cook is by how many cuts and burns they have. The actions that cause those things are incidental to the process, not central, and an excessive amount points to incompetence, not hard and skillful work.

      • KairuByte@lemmy.dbzer0.com
        link
        fedilink
        arrow-up
        53
        ·
        2 days ago

        And yet I experience it so often. That or “effort points” as the metric being used to determine who all stars are.

        Either as a metric just encourages gaming of the system:

        • Why write one line when I can write the same thing in 20?
        • Why take this one effort point task I think will take three when I can just skip it and grab these one effort points I think will take 20 minutes?

        I’ve been on teams that on the surface didn’t have these metrics matter, but the top effort points achiever got bonuses on the DL.

        • Valmond@lemmy.dbzer0.com
          link
          fedilink
          arrow-up
          27
          ·
          2 days ago

          What did you do?? You refacted the code and now it’s better organized but you overall got rid of lines?

          I’ll set up a PMD meeting to help you out of this problem, but fair to say don’t expect a raise or a bonus this year.

      • ViatorOmnium@piefed.social
        link
        fedilink
        English
        arrow-up
        16
        arrow-down
        1
        ·
        2 days ago

        I wouldn’t say PR size is a bad metric, you usually just need yo read it the opposite of how sloppers do it, i.e. the most productive PRs are short and focused.

        • Slotos@feddit.nl
          link
          fedilink
          arrow-up
          2
          ·
          1 day ago

          I’m working with a legacy codebase for the last few months, where a simple PR often ends up crossing a 1000 lines count due to testing and commenting, and I can’t stop apologizing for those.

          Yet there are people out there bragging about 10x changesets.

        • Ethan
          link
          fedilink
          English
          arrow-up
          5
          ·
          2 days ago

          PR size is an awful metric. The bigger the PR, the less reviewable it is.

            • Ethan
              link
              fedilink
              English
              arrow-up
              4
              ·
              2 days ago

              PR size is still an awful metric. It should be within bounds but it should never be an actual metric. Treating it as a metric is an idiotic idea.

          • ViatorOmnium@piefed.social
            link
            fedilink
            English
            arrow-up
            18
            ·
            2 days ago

            I’m not saying it’s a good individual metric. In fact, applying individual metrics to developers (or most workers really), will only land you in Goodhart’s hell.

            But as part of holistic operational health tracking, it’s a useful team level metric, as there is ample evidence that shorter PRs tend to result in less operational issues. And, of course, this is only valid if you don’t try to tie financial rewards to it, otherwise people will forget that PR size is a proxy measure for how easy changes are to review and rollback.

          • KairuByte@lemmy.dbzer0.com
            link
            fedilink
            arrow-up
            1
            ·
            2 days ago

            Yeah, I’ve been guilty of coming up with “cute” solutions that are extremely optimized and concise, but you needed to take a hundred times as long to work through what was going on.

            Usually I would put an explanation comment, but sometimes a less optimized solution is the better option for readability sake.

          • Shayeta@feddit.org
            link
            fedilink
            English
            arrow-up
            1
            ·
            2 days ago

            Hah, if those pesky devs think that they can play the system by just rolling up the code into a single line they got another thing coming - we’re actually tracking PR character count, NOT LOC like some other companies!

      • CorrectAlias@piefed.blahaj.zone
        link
        fedilink
        English
        arrow-up
        45
        ·
        2 days ago

        In college, on the first day of orientation, someone in my class bragged that they wrote 50,000 lines of code for a game that was similar to tic tac toe, emphasizing that he “wrote a lot of code”. A TA told him that it wasn’t a sign that his program was decent and that it really didn’t seem like it should take 50k lines of code to make something as simple as his game.

        He dropped out after the first week of intro to programming.

    • Ech@lemmy.ca
      link
      fedilink
      arrow-up
      4
      ·
      2 days ago

      Might as well be bragging about pictures they’ve taken of their bowel movements.

      • entwine
        link
        fedilink
        arrow-up
        2
        arrow-down
        1
        ·
        2 days ago

        No, this is the most serious subreddit in all of reddit.

    • Klear@quokk.au
      link
      fedilink
      English
      arrow-up
      47
      ·
      2 days ago

      I’m just a hobbyist, but I’m always more proud of commits that remove stuff.

      • traxex@lemmy.dbzer0.com
        link
        fedilink
        arrow-up
        30
        ·
        2 days ago

        Removing shit and it still working perfectly the same is absolutely a goal everyone should have. Less code means less to maintain.

      • sqw@lemmy.sdf.org
        link
        fedilink
        English
        arrow-up
        8
        arrow-down
        1
        ·
        2 days ago

        i refactored some web code i wrote ten years ago and it’s more elegant and a tenth the size with new features because of language advancements. feels great.

  • tatterdemalion
    link
    fedilink
    arrow-up
    11
    ·
    2 days ago

    Slop points aside, I found 5.4 to be pretty ass compared to 5.3 codex. Took way longer and wasted more tokens.

    • ulterno
      link
      fedilink
      English
      arrow-up
      4
      arrow-down
      1
      ·
      2 days ago

      more tokens

      = more money for Amazon = more profitssss.

      • Electricd@lemmybefree.net
        link
        fedilink
        arrow-up
        3
        ·
        1 day ago

        I would love to know if this actually makes a difference though haha. I believe it might make a slight one as it could try to rethink about it depending on the model

        I usually just ask it to recheck line by line for bugs with a thinking model and that’s pretty good at solving potential bugs, but they still need to be reviewed obviously. I only use open weight models though so they might be a bit inferior

  • tempest@lemmy.ca
    link
    fedilink
    arrow-up
    64
    arrow-down
    3
    ·
    2 days ago

    You know what’s funny?

    I use AI to develop software. However when I’m looking for libraries to do things if I see a CLAUDE.md file I have to look and see when it was added and hold it against the library if it’s early in the history.

    It’s like prewar steel.

    I also recognize it’s hypocritical.

      • [object Object]@lemmy.world
        link
        fedilink
        arrow-up
        3
        ·
        2 days ago

        Interesting. I’m unfamiliar with the purpose of that file as originally intended, but if I understand it right, it probably can be used to detect PRs with ai-made code. Tell Claude to write a particular string in every PR, and then reject all PRs with that string.

    • Modern_medicine_isnt@lemmy.world
      link
      fedilink
      arrow-up
      6
      ·
      2 days ago

      It’s not hypocritical. Because you use AI to code, you know how easy it is to just let the AI do it’s thing and not check it’s work. It’s almost like a sirens song. So you know the odds that a library that was coded with AI probably wasn’t checked by a human. That’s just called experience.

    • Fushuan [he/him]@lemmy.blahaj.zone
      link
      fedilink
      arrow-up
      7
      ·
      2 days ago

      It’s the difference between checking for questions in stack overflow and implementing solutions VS pasting every SO solution blindly until something works.

      I do use autocomplete and ask plenty questions, sometimes even use an agent so it makes small changes that I then review and test, but I would never commit unchecked changes, and a claude.md implies that the AI is coding AND committing without supervision.

      I can’t stress enough how different those scenarios are.

  • drath@lemmy.world
    link
    fedilink
    arrow-up
    50
    arrow-down
    4
    ·
    2 days ago

    Positive diff? Pfft, amateurs. If I ever see even a 1000 line PR I’m instantly rejecting and closing it. Learn to code, not generate bullshit.

  • MortUS@lemmy.world
    link
    fedilink
    English
    arrow-up
    18
    ·
    2 days ago

    I like that this is lowkey a Polymarket Advertisement too. The internet truly is a wonderous place.