Merge branch 'generative-computing:main' into main

Author: tuliocoppola

docs/examples/generative_slots/inter_module_composition/summarizers.py

@@ -13,4 +13,4 @@ def summarize_contract(contract_text: str) -> str:
 
 @generative
 def summarize_short_story(story: str) -> str:
-    """Summarize a short story, with one paragraph on plot and one paragraph on braod themes."""
+    """Summarize a short story, with one paragraph on plot and one paragraph on broad themes."""

docs/examples/notebooks/compositionality_with_generative_slots.ipynb

@@ -95,7 +95,7 @@
     "\n",
     "@generative\n",
     "def summarize_short_story(story: str) -> str:\n",
-    "    \"\"\"Summarize a short story, with one paragraph on plot and one paragraph on braod themes.\"\"\""
+    "    \"\"\"Summarize a short story, with one paragraph on plot and one paragraph on broad themes.\"\"\""
    ]
   },
   {

docs/examples/notebooks/document_mobject.ipynb

@@ -60,7 +60,7 @@
    },
    "outputs": [],
    "source": [
-    "!uv pip install mellea -q"
+    "!uv pip install mellea[docling]"
    ]
   },
   {

docs/examples/notebooks/georgia_tech.ipynb

@@ -14,7 +14,7 @@ def summarize_contract(contract_text: str) -> str:
 
 @generative
 def summarize_short_story(story: str) -> str:
-    """Summarize a short story, with one paragraph on plot and one paragraph on braod themes."""
+    """Summarize a short story, with one paragraph on plot and one paragraph on broad themes."""
 
 
 # The Decision Aides Library

docs/examples/tutorial/compositionality_with_generative_slots.py

@@ -3,6 +3,7 @@
 import inspect
 import re
 from collections.abc import Callable
+from copy import copy
 from typing import Any, overload
 
 from mellea.backends import (
@@ -97,6 +98,9 @@ def __init__(
         self.validation_fn = validation_fn
         self.check_only = check_only
 
+        # Used for validation. Do not manually populate.
+        self._output: str | None = None
+
     def validate(
         self,
         backend: Backend,
@@ -117,17 +121,18 @@ def validate(
             assert isinstance(last_output, ModelOutputThunk), (
                 " Context has no appropriate last output"
             )
-            self._output = last_output.value  # type: ignore
+
+            # Create a copy of the requirement that holds the output
+            # and its template gets populated with the output correctly.
+            req_copy = copy(self)
+            req_copy._output = last_output.value
             llm_as_a_judge_result = backend.generate_from_context(
-                self,
+                req_copy,
                 ctx,
                 format=format,
                 model_options=model_options,
                 generate_logs=generate_logs,
             )
-            # This is crucial, because requirements can get reused;
-            # this also means requirements are not thread-safe.
-            self._output = None
             return ValidationResult(
                 result=self.output_to_bool(llm_as_a_judge_result),
                 reason=llm_as_a_judge_result.value,

mellea/stdlib/requirement.py

@@ -138,6 +138,7 @@ ignore = [
 #   "UP006",  # List vs list, etc
 #   "UP007",  # Option and Union
 #   "UP035",  # `typing.Set` is deprecated, use `set` instead"
+  "PD901",  # Avoid using the generic variable name `df` for DataFrames 
 ]
 
 [tool.ruff.lint.pydocstyle]
@@ -151,7 +152,7 @@ combine-as-imports = true
 split-on-trailing-comma = false
 
 [tool.codespell]
-ignore-words-list = 'mellea,hashi,noo,Asai,asai'
+ignore-words-list = 'mellea,hashi,noo,Asai,asai,nd'
 check-filenames = true
 check-hidden = false
 regex = "(?<![a-z])[a-z'`]+|[A-Z][a-z'`]*|[a-z]+'[a-z]*|[a-z]+(?=[_-])|[a-z]+(?=[A-Z])|\\d+"