• lib1 [comrade/them]@hexbear.net
      link
      fedilink
      English
      arrow-up
      13
      ·
      edit-2
      1 day ago

      Kevlin Henney did a presentation which included a code review of the left pad package. Several of his unit tests failed.

      The code in question wasn’t particularly well written or anything
      module.exports = leftpad;
      
      function leftpad (str, len, ch) {
          str = String(str);
      
          var i = -1;
      
          ch || (ch = ' ');
          len = len - str.length;
      
      
          while (++i < len) {
              str = ch + str;
          }
      
          return str;
      }
      
      And this is the package that it was replaced with. It also failed several tests
      'use strict';
      module.exports = leftPad;
      
      var cache = [
        '',
        ' ',
        '  ',
        '   ',
        '    ',
        '     ',
        '      ',
        '       ',
        '        ',
        '         '
      ];
      
      function leftPad (str, len, ch) {
        // convert `str` to a `string`
        str = str + '';
        // `len` is the `pad`'s length now
        len = len - str.length;
        // doesn't need to pad
        if (len <= 0) return str;
        // `ch` defaults to `' '`
        if (!ch && ch !== 0) ch = ' ';
        // convert `ch` to a `string` cuz it could be a number
        ch = ch + '';
        // cache common use cases
        if (ch === ' ' && len < 10) return cache[len] + str;
        // `pad` starts with an empty string
        var pad = '';
        // loop
        while (true) {
          // add `ch` to `pad` if `len` is odd
          if (len & 1) pad += ch;
          // divide `len` by 2, ditch the remainder
          len >>= 1;
          // "double" the `ch` so this operation count grows logarithmically on `len`
          // each time `ch` is "doubled", the `len` would need to be "doubled" too
          // similar to finding a value in binary search tree, hence O(log(n))
          if (len) ch += ch;
          // `len` is 0, exit the loop
          else break;
        }
        // pad `str`!
        return pad + str;
      }
      
      • chgxvjh [he/him, comrade/them]@hexbear.net
        link
        fedilink
        English
        arrow-up
        18
        ·
        1 day ago

        Not really caring about things working correctly didn’t start with LLM coding assistants/agents. Outside of a few industries programming never was a serious engineering discipline and people mostly rejected the idea that it should be one.

        • SwitchyandWitchy [she/her]@hexbear.net
          link
          fedilink
          English
          arrow-up
          12
          ·
          edit-2
          1 day ago

          brogrammers rejected it. Nerds like Margaret Hamilton stood on principle

          This caption is misleading btw. She never worked directly for NASA, she worked at MIT who were contracted by NASA for the software.

          I also know people who have worked in nuclear energy and aviation software and they are really serious about not taking shortcuts. And regularly complain about higher ups saying things and occasionally pressuring them against industry standards in the name of speeding things up.

            • SwitchyandWitchy [she/her]@hexbear.net
              link
              fedilink
              English
              arrow-up
              3
              ·
              22 hours ago

              For some of the really flight critical stuff that I picked their brains on, the on-cpu cache was disabled because using it would make deterministally analyzing the executive time too difficult. There also wasn’t a pre-emptive multitasking scheduler but instead every task would run during a scheduled time slot in a big superloop.

              All of this caution actually made the software a lot more primitive than the software that Margaret Hamilton led the development of for the Apollo program, but these days cpu cycles and memory are a lot cheaper than the engineering time to implement all these things in a safe way, or so they said.

        • SoyViking [he/him]@hexbear.net
          link
          fedilink
          English
          arrow-up
          9
          ·
          1 day ago

          Clients want new shiny features and they want them fast and they want them cheap and they want to change their scope 20 things during the project. They do not want to pay for boring unit tests that confirms that the system works.