• muntedcrocodile
    link
    fedilink
    English
    arrow-up
    8
    arrow-down
    8
    ·
    1 month ago

    I dont think comparisons should be doing type conversion if i compare a float to an int i want it to say false cos types are different.

    • Blackmist@feddit.uk
      link
      fedilink
      English
      arrow-up
      19
      ·
      edit-2
      1 month ago

      I don’t think that’s how most programmers expect it to work at all.

      However most people would also expect 0.1+0.2==0.3 to return true, so what do I know.

      Floating point is something most of us ignore until it bites us in the ass. And then we never trust it again.

      • thebestaquaman@lemmy.world
        link
        fedilink
        English
        arrow-up
        8
        arrow-down
        1
        ·
        1 month ago

        I have to admit: If you (semi-)regularly use floating point comparisons in programming, I don’t know why you would ever expect 0.1 + 0.2 == 0.3 to return true. It’s common practice to check abs(a - b) < tol, where tol is some small number, to the point that common unit-testing libraries have built-in methods like assertEqual(a, b, tol) specifically for checking whether floats are “equal”.

        • Pelicanen@sopuli.xyz
          link
          fedilink
          English
          arrow-up
          3
          ·
          30 days ago

          Yeah, a lot of editors throw warnings for using the equals operator with floats by default, as far as I know it’s considered bad practice to do it that way.

        • Blackmist@feddit.uk
          link
          fedilink
          English
          arrow-up
          2
          ·
          29 days ago

          The issue is a lot of people use floating point numbers, but don’t even know it.

          How many programmers right now are using JS, the most popular language in the world? How many of them do you think understand floating point numbers and their theoretical levels of accuracy? How many of them are unknowingly using floating points to store currency values?

          How many of them could accurately predict the result of the following?

          • 2.99+1.52==4.51
          • 2.99+1.53==4.52
          • 2.99+1.54==4.53

          Now imagine that as code to make sure you’ve paid the right amount in an online store. I guarantee you there is code out there right now that won’t let you finish a sale if the total of the basket adds up a certain way.

      • muntedcrocodile
        link
        fedilink
        English
        arrow-up
        3
        ·
        1 month ago

        Thats why i recon its good to keep u aware of it. Mind u i find its often fine as long as my ide and chagpt know what type it is im usually fine.

        I do kinda like the rigidity of types tho. Proper Python type hints are a godsend.

      • Miaou@jlai.lu
        link
        fedilink
        English
        arrow-up
        2
        arrow-down
        1
        ·
        1 month ago

        Then most people shouldn’t be writing code, I don’t know what else to tell you, this is probably one of the first thing you learn about FP arithmetic, and any decent compiler/linter should warn you about that.

    • jdnewmil@lemmy.ca
      link
      fedilink
      English
      arrow-up
      5
      ·
      1 month ago

      Way too late for that. Every language I know makes some kind of auto conversion for numeric comparisons… and sometimes for strings as well.

      • Ephera@lemmy.ml
        link
        fedilink
        English
        arrow-up
        5
        ·
        edit-2
        1 month ago

        I know of Rust, which is pedantic enough to not allow comparing integers to floats directly.

        In certain situations, it even disallows making assumptions about equality and ordering between floats.

        • micka190@lemmy.world
          link
          fedilink
          English
          arrow-up
          2
          ·
          1 month ago

          In certain situations, it even disallows making assumptions about equality and ordering between floats.

          I’m guessing it does this when you define both floats in the same location with constant values.

          The correct way to compare floats whose values you don’t know ahead of time is to compare the absolute of their delta against a threshold.

          i.e.

          abs(a - b) <= 0.00001

          The idea being that you can’t really compare floats due to how they work. By subtracting them, you can make sure the values are “close enough” and avoid issues with precision not actually mattering all that much past the given threshold. If you define 2 constant values, though, the compiler can probably simplify it for you and just say “Yeah, these two should be the same value at runtime”.

          • Ephera@lemmy.ml
            link
            fedilink
            English
            arrow-up
            3
            ·
            1 month ago

            Unfortunately, that’s not what it’s about.

            With Rust’s language design¹, one could easily forbid ever comparing two floats via the normal operators (not just for literals), but they decided to allow that nonetheless. I’m guessing, because it would just be too annoying for generic implementations, if you’d always need special treatment for floats.

            Rather, what I was referring to, is that they’ve split partial ordering and partial equality from their total variants. And integers are marked that they can do total equality and total ordering, whereas floats are not.

            Honestly, it doesn’t come up a lot, but I know for example, if you want to use something as a key in a HashMap, it needs to have total equality.
            And in a BTreeMap (basically a binary tree, i.e. keys are inserted in a sorted manner), total ordering is required for the keys.

            ¹) Basically, comparison is typed and each type needs to opt into comparison with any type it wants to be compared to. So, if you define a new data type, then by default you cannot ask it whether it’s equal to or less/more than anything else.

          • Miaou@jlai.lu
            link
            fedilink
            English
            arrow-up
            1
            ·
            edit-2
            1 month ago

            Rust has a warning (has it been promoted to error? I think it was supposed to be) about comparing floats. Nothing to do with same being const. You basically don’t have an equality operator for them

        • muntedcrocodile
          link
          fedilink
          English
          arrow-up
          2
          ·
          1 month ago

          I still cant properly manage my head around the rust object borrowing. My ray tracer implementation from that blog on ray tracing was slow as shiiiit.

          • Ephera@lemmy.ml
            link
            fedilink
            English
            arrow-up
            5
            ·
            1 month ago

            Not sure, what blog post you’re talking about, but there’s only really three things you can be doing wrong:

            • Tons of cloning.
            • Running your application from a debug build rather than release build.
            • All the usual things one can be doing wrong in any programming language. Like, I imagine real-world raytracing is done on the GPU and uses highly optimized algorithms. I doubt a blog post would dive into those depths. And well, any kind of graphics programming is extremely slow, if you don’t issue the exact right incantations that the GPU manufacturer optimized for.
    • Womble@lemmy.world
      link
      fedilink
      English
      arrow-up
      4
      ·
      1 month ago

      But how far should that be taken should 8 == 8 return false because one is an unsigned int and the other is signed? Or 0.0 == 0.0 where they are floats and doubles? You can make a case for those due to a strong type system but it would go against most peoples idea of what equality is.

      • muntedcrocodile
        link
        fedilink
        English
        arrow-up
        1
        ·
        30 days ago

        If bits aren’t same then i dont want it to tell me they are the same. And python just has one implementation for int and float.

        I like python cos everything’s an object i dont want different types of objects to evaluate the same they are fundamentally different objects is that not what u would expect?

        • Womble@lemmy.world
          link
          fedilink
          English
          arrow-up
          2
          ·
          29 days ago

          Even in python you can have control of what types of numbers are used under the hood with numpy arrays (and chances are if you are using floats in any quantity you want to be using numpy). I would be very surprised if array([1,2,3], dtype=uint8) == array([1,2,3], dtype=int16) gave [False, False, False]. In general I think == for numbers should give mathematical equivalence, with the understanding that comparing floats is highly likely to give false negatives unless you are extremely careful with what you are comparing.

            • Womble@lemmy.world
              link
              fedilink
              English
              arrow-up
              2
              ·
              edit-2
              29 days ago

              More Fortran than C, but its the same for any language doing those sorts of array mathematics, they will be calling compiled versions of blas and lapack. Numpy builds up low level highly optimised compiled functions into a coherant python ecosystem. A numpy array is a C array with some metadata sure, but a python list is also just a C array of pointers to pyobjects.

    • Sylvartas@lemmy.world
      link
      fedilink
      English
      arrow-up
      4
      ·
      1 month ago

      That makes sense, but then you’d just have people converting the int to a float manually and run into the exact same issues.

      • muntedcrocodile
        link
        fedilink
        English
        arrow-up
        5
        ·
        1 month ago

        I mean honestly its expected u should check floats similarity not equivalence.

        • Sylvartas@lemmy.world
          link
          fedilink
          English
          arrow-up
          5
          arrow-down
          1
          ·
          1 month ago

          Agreed. But the less experienced programmers I know are surprisingly naive about this.

          • muntedcrocodile
            link
            fedilink
            English
            arrow-up
            3
            ·
            1 month ago

            Yeah its gonna be one of those problems chatgpt ain’t gonna help

      • Miaou@jlai.lu
        link
        fedilink
        English
        arrow-up
        2
        ·
        1 month ago

        They wouldn’t be running into an issue, but creating one, that’s different

        • Sylvartas@lemmy.world
          link
          fedilink
          English
          arrow-up
          1
          ·
          1 month ago

          Meh. Imo anyone comparing an integer to a float and not expecting one of them to be implicitly casted to the other’s type will create that issue for themselves when doing the same thing with an explicit cast.

          • Miaou@jlai.lu
            link
            fedilink
            English
            arrow-up
            3
            ·
            30 days ago

            What I meant is, the former can be a genuine mistake, the latter is a conscious (probably uneducated) decision

    • Miaou@jlai.lu
      link
      fedilink
      English
      arrow-up
      3
      arrow-down
      1
      ·
      1 month ago

      Idiots downvoting you (c/technology…) but this how e.g. Haskell and rust handle that, and probably most strongly typed languages

      • muntedcrocodile
        link
        fedilink
        English
        arrow-up
        1
        ·
        30 days ago

        Yeah i think python needs stronger type enforcement i do like how they have been improving type hinting tho really helps with preventing bugs