Resolve PR comments

lihuoran · lihuoran · commit f96c4d2879bb · 2022-08-31T14:39:53.000+08:00
diff --git a/maro/common.py b/maro/common.py
@@ -9,10 +9,10 @@ class BaseDecisionEvent:
     - CascadeEvent & AtomEvent: used to drive the MARO Env / business engine.
     - DecisionEvent: exposed to users as a means of communication.
 
-    The latter one serves the `payload` of the former ones inside of MARO Env.
+    The latter one serves as the `payload` of the former ones inside of MARO Env.
 
     Therefore, the related namings might be a little bit tricky.
-    - Inside MARO Env: `decision_event` is actually a CascadeEvent or AtomEvent. DecisionEvent is the payload of them.
+    - Inside MARO Env: `decision_event` is actually a CascadeEvent. DecisionEvent is the payload of them.
     - Outside MARO Env (for users): `decision_event` is a DecisionEvent.
     """
 
diff --git a/maro/event_buffer/event_buffer.py b/maro/event_buffer/event_buffer.py
@@ -137,7 +137,7 @@ def gen_cascade_event(self, tick: int, event_type: object, payload: object) -> C
         Returns:
             CascadeEvent: Cascade event object.
         """
-        return cast(CascadeEvent, self._event_pool.gen(tick, event_type, payload, is_cascade=False))
+        return cast(CascadeEvent, self._event_pool.gen(tick, event_type, payload, is_cascade=True))
 
     def gen_decision_event(self, tick: int, payload: BaseDecisionEvent) -> CascadeEvent:
         """Generate a decision event that will stop current simulation, and ask agent for action.
diff --git a/maro/simulator/core.py b/maro/simulator/core.py
@@ -96,7 +96,7 @@ def step(
         """Push the environment to next step with action.
 
         Under Sequential mode:
-            - If `action` is None, a empty list will be assigned to the decision event.
+            - If `action` is None, an empty list will be assigned to the decision event.
             - Otherwise, the action(s) will be assigned to the decision event.
 
         Under Joint mode:
@@ -120,7 +120,7 @@ def step(
             since the latter one will assign the n actions to the first n decision events.
 
         Args:
-            action (Union[BaseAction, List[ActionPayload], None]): Action(s) from agent.
+            action (Union[BaseAction, List[BaseAction], None]): Action(s) from agent.
 
         Returns:
             tuple: a tuple of (metrics, decision event, is_done).
