BUG: Python 3.11 requires random.sample(_sequence_, int)

Leengit · Leengit · commit 8b3bab7febed · 2024-09-17T12:33:15.000-04:00
diff --git a/histomics_stream/configure.py b/histomics_stream/configure.py
@@ -589,7 +589,7 @@ def __call__(self, slide):
         if 0 <= self.randomly_select < len(slide["tiles"]):
             # Choose a subset of the tiles randomly
             slide["tiles"] = dict(
-                random.sample(slide["tiles"].items(), self.randomly_select)
+                random.sample(sorted(slide["tiles"].items()), self.randomly_select)
             )
 
     def check_mask_filename(self, mask_filename):
@@ -806,7 +806,7 @@ def __call__(self, slide):
         if 0 <= self.randomly_select < len(slide["tiles"]):
             # Choose a subset of the tiles randomly
             slide["tiles"] = dict(
-                random.sample(slide["tiles"].items(), self.randomly_select)
+                random.sample(sorted(slide["tiles"].items()), self.randomly_select)
             )
 
 

Original file line number	Diff line number	Diff line change
`@@ -589,7 +589,7 @@ def __call__(self, slide):`
`589`	`589`	`if 0 <= self.randomly_select < len(slide["tiles"]):`
`590`	`590`	`# Choose a subset of the tiles randomly`
`591`	`591`	`slide["tiles"] = dict(`
`592`		`- random.sample(slide["tiles"].items(), self.randomly_select)`
	`592`	`+ random.sample(sorted(slide["tiles"].items()), self.randomly_select)`
`593`	`593`	`)`
`594`	`594`
`595`	`595`	`def check_mask_filename(self, mask_filename):`
`@@ -806,7 +806,7 @@ def __call__(self, slide):`
`806`	`806`	`if 0 <= self.randomly_select < len(slide["tiles"]):`
`807`	`807`	`# Choose a subset of the tiles randomly`
`808`	`808`	`slide["tiles"] = dict(`
`809`		`- random.sample(slide["tiles"].items(), self.randomly_select)`
	`809`	`+ random.sample(sorted(slide["tiles"].items()), self.randomly_select)`
`810`	`810`	`)`
`811`	`811`
`812`	`812`