Merge pull request #1430 from toslunar/rand-interval

Make the method `cupy.random.RandomState.interval` private

cupy/random/generator.py

@@ -182,7 +182,7 @@ def random_sample(self, size=None, dtype=float):
         RandomState._1m_kernel(out)
         return out
 
-    def interval(self, mx, size):
+    def _interval(self, mx, size):
         """Generate multiple integers independently sampled uniformly from ``[0, mx]``.
 
         Args:
@@ -202,7 +202,7 @@ def interval(self, mx, size):
             a ``numpy.uint32`` array is returned.
         """
         if size is None:
-            return self.interval(mx, 1).reshape(())
+            return self._interval(mx, 1).reshape(())
         elif isinstance(size, int):
             size = (size, )
 
@@ -490,7 +490,7 @@ def randint(self, low, high=None, size=None, dtype='l'):
             raise NotImplementedError(
                 'Sampling from a range whose extent is larger than int32 '
                 'range is currently not supported')
-        x = self.interval(diff, size).astype(dtype, copy=False)
+        x = self._interval(diff, size).astype(dtype, copy=False)
         cupy.add(x, lo, out=x)
         return x
 

tests/cupy_tests/random_tests/test_generator.py

@@ -258,7 +258,7 @@ def test_randn_invalid_argument(self):
 @testing.gpu
 class TestInterval(RandomGeneratorTestCase):
 
-    target_method = 'interval'
+    target_method = '_interval'
 
     def test_zero(self):
         shape = (2, 3)