lint, comments, removing dead code, fixing typos

PiperOrigin-RevId: 588369744
Change-Id: Ib68335add19e58b394da4d4d3825412df506e3f6

concordia/associative_memory/associative_memory.py

@@ -116,7 +116,7 @@ def get_data_frame(self):
       return self._memory_bank.copy()
 
   def _get_top_k_cosine(self, x: np.ndarray, k: int):
-    """Returns the top k rows of a dataframe that have the highest cosine similarity to an input vector x.
+    """Returns the top k most cosine similar rows to an input vector x.
 
     Args:
       x: The input vector.
@@ -139,7 +139,7 @@ def _get_top_k_cosine(self, x: np.ndarray, k: int):
   def _get_top_k_similar_rows(
       self, x, k: int, use_recency: bool = True, use_importance: bool = True
   ):
-    """Returns the top k rows of a dataframe that have the highest cosine similarity to an input vector x.
+    """Returns the top k most similar rows to an input vector x.
 
     Args:
       x: The input vector.

concordia/components/agent/plan.py

@@ -82,7 +82,7 @@ def _log(self, entry: str):
   def observe(self, observation: str):
     self._last_observation.append(observation)
 
-  def update(self, push_to_mem=True):
+  def update(self):
     observation = '\n'.join(self._last_observation)
     self._last_observation = []
     memories = self._memory.retrieve_associative(

concordia/components/sequential.py

@@ -44,9 +44,9 @@ def observe(self, observation: str):
     for comp in self._components:
       comp.observe(observation)
 
-  def update_before_event(self, event_statement: str) -> None:
+  def update_before_event(self, cause_statement: str) -> None:
     for comp in self._components:
-      comp.update_before_event(event_statement)
+      comp.update_before_event(cause_statement)
 
   def update_after_event(self, event_statement: str) -> None:
     for comp in self._components:

examples/phone/components/scene.py

@@ -15,7 +15,6 @@
 """A GameMaster that simulates a player's interaction with their phone."""
 
 import textwrap
-
 from concordia.agents import basic_agent
 from concordia.associative_memory import blank_memories
 from concordia.clocks import game_clock

examples/phone/components/triggering.py

@@ -16,7 +16,6 @@
 """A component that runs the phone scene when a phone action is detected."""
 
 from collections.abc import Sequence
-
 from concordia.agents import basic_agent
 from concordia.associative_memory import associative_memory
 from concordia.associative_memory import blank_memories

examples/village/components/elections.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 
-"""Construct and an externality that implements elections within a game master."""
+"""Component that implements elections within a game master."""
 
 from collections.abc import Callable, Sequence
 import datetime