Skip to content

Commit

Permalink
Remove potentially expensive check in ForestMerge.
Browse files Browse the repository at this point in the history
  • Loading branch information
riga committed Jun 10, 2024
1 parent d565371 commit f295d05
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions law/contrib/tasks/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -210,10 +210,6 @@ def __init__(self, *args, **kwargs):
self._merge_forest = None
self._leaves_per_tree = None

# the merge factor should not be 1
if self.merge_factor == 1:
raise ValueError("merge factor must not be 1")

# modify_param_values prevents the forest from being a workflow, but still check
if self.is_forest() and self.is_workflow():
raise Exception("merge forest must not be a workflow, {} misconfigured".format(self))
Expand Down

0 comments on commit f295d05

Please sign in to comment.