Merge pull request #2413 from OpenMined/issue-2412

Closes #2412, remove simplification and detailing for plans

Author: iamtrask

examples/tutorials/Part 8 - Introduction to Plans.ipynb

@@ -819,4 +819,4 @@
  },
  "nbformat": 4,
  "nbformat_minor": 2
-}
+}

syft/federated/plan.py

@@ -624,9 +624,8 @@ def simplify(plan: "Plan") -> tuple:
             tuple: a tuple holding the unique attributes of the Plan object
 
         """
-        readable_plan = sy.serde._simplify(plan.readable_plan)
         return (
-            readable_plan,
+            plan.readable_plan,  # We're not simplifying because readable_plan is already simplified
             sy.serde._simplify(plan.id),
             sy.serde._simplify(plan.arg_ids),
             sy.serde._simplify(plan.result_ids),
@@ -656,7 +655,7 @@ def detail(worker: AbstractWorker, plan_tuple: tuple) -> "Plan":
             id=id,
             arg_ids=arg_ids,
             result_ids=result_ids,
-            readable_plan=sy.serde._detail(worker, readable_plan),
+            readable_plan=readable_plan,  # We're not detailing, see simplify() for details
             is_built=is_built,
         )