Ensure that complement offset is always integer (renatahodovan#35)

Use integer division when computing its value instead of casting it to int when used.
akosthekiss · Jul 14, 2023 · 63668bb · 63668bb
1 parent ef59596
commit 63668bb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/picire/dd.py b/picire/dd.py
@@ -88,7 +88,7 @@ def __call__(self, config):
                 elif len(subsets) < len(config):
                     # No interesting configuration is found but it is still not the finest splitting, start new iteration.
                     next_subsets = self._split(subsets)
-                    complement_offset = (complement_offset * len(next_subsets)) / len(subsets)
+                    complement_offset = (complement_offset * len(next_subsets)) // len(subsets)
                     subsets = next_subsets
 
                     logger.info('\tIncreased granularity')
@@ -121,7 +121,7 @@ def _reduce_config(self, run, subsets, complement_offset):
                 config_id = (f'r{run}', f's{i}')
                 config_set = subsets[i]
             else:
-                i = int((-i - 1 + complement_offset) % n)
+                i = (-i - 1 + complement_offset) % n
                 config_id = (f'r{run}', f'c{i}')
                 config_set = [c for si, s in enumerate(subsets) for c in s if si != i]
                 i = -i - 1

diff --git a/picire/parallel_dd.py b/picire/parallel_dd.py
@@ -79,7 +79,7 @@ def _reduce_config(self, run, subsets, complement_offset):
                 config_id = (f'r{run}', f's{i}')
                 config_set = subsets[i]
             else:
-                i = int((-i - 1 + complement_offset) % n)
+                i = (-i - 1 + complement_offset) % n
                 config_id = (f'r{run}', f'c{i}')
                 config_set = [c for si, s in enumerate(subsets) for c in s if si != i]
                 i = -i - 1