search

KavrakiLab / robowflex

Making MoveIt Easy!
https://kavrakilab.github.io/robowflex/
Other
114 stars 24 forks source link

UniformInt does has wrong range [BUG] #294

Closed ChamzasKonstantinos closed 1 year ago

ChamzasKonstantinos commented 2 years ago

function RandomInt() returns a value in [lower_bound, upper_bound] instead of [lower_bound, upper_bound) as mentioned here.

It's an easy fix, but the question is, do we fix the documentation or the function? I propose the function since it works better with vector indexing.

zkingston commented 2 years ago

Sure, we should fix the function to return in the [) interval rather than [].