slient overflow occurs in tf.range leading to incorrect result, here is a possible fix about tensorflow HOT 6 OPEN

Hi @QuantumCoder4 , @Aloqeely

I acknowledge the issue of overflow and attached gist for reference.The proposed fix might work it seems. Please feel free to raise a PR.

from tensorflow.

Your analysis and solution seem valid. Open a PR if you'd like, otherwise I can handle it

from tensorflow.

I am afraid I don't have time in these few days, I would appreciate it if you can help me to open one.

from tensorflow.

Hi @Aloqeely ,

I have a question. What happens if the value of delta =1 here keeping other two params same. It still might cause overflow ?

from tensorflow.

Line 83 in sequence_ops.cc:

if (delta > 0) {
      OP_REQUIRES(
          context, start <= limit,
          errors::InvalidArgument(
              "Requires start <= limit when delta > 0: ", start, "/", limit));
    }

In that case, this if statement would get triggered since delta = 1, and then start <= limit would get evaluated to false so the function will raise an error

from tensorflow.

The proposed fix above won't work - it leads to different behavior. See the discussion in the (now closed) PR #64144.

Someone can pick that up if they like.

from tensorflow.