• Gxost@lemmy.world
    link
    fedilink
    arrow-up
    36
    arrow-down
    3
    ·
    13 hours ago

    Nah, I was excited to read about the algorithmic change, but it turned out to be an obvious change. I would replace nested loops with a map too. The result is impressive, though.

    • drspod@lemmy.ml
      link
      fedilink
      arrow-up
      15
      ·
      11 hours ago

      Marketing departments love to make a huge deal out of this kind of thing, because they only see the big number improvement and don’t really understand that this was just some dev’s Wednesday afternoon.

      • mamotromico@lemmy.ml
        link
        fedilink
        arrow-up
        2
        ·
        55 minutes ago

        I mean, it’s still really impressive upgrade even if technically it was a simple change, they are right to make a fuss about the change

      • abbadon420@lemm.ee
        link
        fedilink
        arrow-up
        25
        ·
        11 hours ago

        And they are right to do so. In the grand scheme of things, it doesn’t really matter how much time you spend on a problem. It’s the result that matters. I remember a meme where a dev would place a “wait” function in a new feature. Than remove the wait call and call it a free update and get lots of praise from the customer.

  • manxu@piefed.social
    link
    fedilink
    English
    arrow-up
    10
    arrow-down
    3
    ·
    14 hours ago

    we traced the issue to a 15-year-old Git function with O(N²) complexity and fixed it with an algorithmic change, reducing backup times exponentially.

    I feel like there is something wrong with this sentence.

    • _taem@discuss.tchncs.de
      link
      fedilink
      arrow-up
      14
      ·
      12 hours ago

      I’m not a native speaker, but would agree that it sounds imprecise. To my understanding, that’s a polynomial reduction of the time (O(n^2) to O(n): quadratic to linear) and not an exponential speed-up (O(2^n) to O(n): exponential to linear). 🤷 Colloquially, “exponentially” seems to be used synonymously to “tremendously” or similar.

      • Giooschi@lemmy.world
        link
        fedilink
        English
        arrow-up
        6
        ·
        9 hours ago

        and not an exponential speed-up (O(2^n) to O(n): exponential to linear)

        Note that you can also have an exponential speed-up when going from O(n) (or O(n^2) or other polynomial complexities) to O(log n). Of course that didn’t happen in this case.

    • drspod@lemmy.ml
      link
      fedilink
      arrow-up
      10
      arrow-down
      2
      ·
      11 hours ago

      They make the same mistake further down the article:

      However, the implementation of the command suffered from poor scalability related to reference count, creating a performance bottleneck. As repositories accumulated more references, processing time increased exponentially.

      This article writer really loves bullet point lists, too. 🤨

      • drspod@lemmy.ml
        link
        fedilink
        arrow-up
        0
        ·
        7 hours ago

        On a technical blog post by a software company about the details of solving an algorithmic complexity problem?

        Careless, and showing that the author does not understand technical communication, where precision is of great importance.

        • FizzyOrange@programming.dev
          link
          fedilink
          arrow-up
          2
          arrow-down
          1
          ·
          2 hours ago

          This is fine precisely because it is a blog post. If it was a scientific paper… sure maybe they shouldn’t say that. But the meaning is abundantly clear from the context. There is no ambiguity.

    • Deebster@infosec.pub
      link
      fedilink
      arrow-up
      15
      arrow-down
      3
      ·
      edit-2
      14 hours ago

      Seem ok to me, both in grammar and what it’s saying about the change. O(N²) to O(N) would be an exponential drop (2 down to 1, in fact).

      • Giooschi@lemmy.world
        link
        fedilink
        English
        arrow-up
        6
        ·
        9 hours ago

        An “exponential drop” would be a drop that follow an exponential curve, but this doesn’t. What you mean is a “drop in the exponent”, which however doesn’t sound as nice.

      • Bogasse@lemmy.ml
        link
        fedilink
        arrow-up
        5
        ·
        10 hours ago

        It’s at least misleading 😛

        But I have to agree that for any non-math people this would convey the right idea, whereas “quadratic improvement” would probably not mean anything 🤷