Announcement

Collapse
No announcement yet.

Please implement Short-Circuit evaluation for Blueprint

Collapse
X
 
  • Filter
  • Time
  • Show
Clear All
new posts

    #91
    Originally posted by franktech View Post
    True but it also says:
    Developer Notes: this issue no longer occurs in 4.16 Main CL 3275234
    Sadly that Note seems outdated https://answers.unrealengine.com/que...-ue-20044.html

    Really was looking forward for that fix...

    Comment


      #92
      Originally posted by franktech View Post
      True but it also says:
      Checked in UE 4.16 - still call every option node function.
      Checked in UE master - still call every option node function.

      Click image for larger version

Name:	472da461003a27a11fc0150b6dbecdd9.png
Views:	1
Size:	71.8 KB
ID:	1128641

      Here is my test code. Number Log outputs formatted string Option {Number} Log. All three are printed.

      Comment


        #94
        +1, here is another case when this would be useful:
        https://forums.unrealengine.com/deve...s-branch-nodes

        Comment


          #95
          +1 here, still waiting for this.. just ran into the same issues again, as i built some blueprint nodes like i would've done using C++. Obviously the access violations spammed my log and i had to adjust my blueprint using much more nodes

          Comment


            #96
            I just assumed that BP did this automatically.

            Comment


              #97
              Not sure how we've managed 7 pages, but there's a reason this feature doesn't exist - it can't. It would totally break recursive functionality.

              Edit: as an aside, talking about proper short circuiting - conditions failing early and logic not being evaluated when it doesn't need to be would be a welcome change (boolean condition checks, select nodes) - but this is unrelated.
              Last edited by ambershee; 01-09-2018, 11:52 AM.

              Comment


                #98
                Originally posted by ambershee View Post
                Not sure how we've managed 7 pages, but there's a reason this feature doesn't exist - it can't. It would totally break recursive functionality.

                Edit: as an aside, talking about proper short circuiting - conditions failing early and logic not being evaluated when it doesn't need to be would be a welcome change (boolean condition checks, select nodes) - but this is unrelated.
                Even if it was a checkbox option on the node or its own node?

                Comment


                  #99
                  +1 for the request.

                  Originally posted by ambershee View Post
                  there's a reason this feature doesn't exist - it can't. It would totally break recursive functionality.
                  how would this break recursive functionality?

                  Comment


                    Originally posted by ambershee View Post
                    Edit: as an aside, talking about proper short circuiting - conditions failing early and logic not being evaluated when it doesn't need to be would be a welcome change (boolean condition checks, select nodes) - but this is unrelated.
                    This is neither an "aside" nor "unrelated", this is exactly what the thread is about, as well as being an example of how terrible people are at interpreting text nowadays.

                    Anyway, the biggest roadblock to implement this is that all pure nodes in BP like AND, OR, and SELECT are actually functions under the hood and thus need all their arguments before they can run. For short circuit evaluation there would need to be a new kind of node that is capable of lazily evaluate its pins, with very different underlying code. Doesn't sound trivial at all.

                    Comment


                      Bump. Desperately need this. Don't want to have so many branches and IsValid macros to write otherwise very normal code.

                      Comment


                        Bump... please make it a priority :')

                        Comment

                        Working...
                        X