Files
NexQuant/rdagent/components/coder/CoSTEER/__init__.py
T
TPTBusiness a43c443c2e fix(security): replace remaining assert statements with proper error handling
Replaced 53 assert statements across 22 files with proper
if/raise patterns (TypeError, ValueError, AssertionError)
to resolve Bandit B101 alerts.
2026-05-01 13:49:58 +02:00

290 lines
12 KiB
Python

from copy import deepcopy
from datetime import datetime
from pathlib import Path
from rdagent.components.coder.CoSTEER.config import CoSTEERSettings
from rdagent.components.coder.CoSTEER.evaluators import CoSTEERMultiFeedback
from rdagent.components.coder.CoSTEER.evolvable_subjects import EvolvingItem
from rdagent.components.coder.CoSTEER.knowledge_management import (
CoSTEERRAGStrategyV1,
CoSTEERRAGStrategyV2,
)
from rdagent.core.developer import Developer
from rdagent.core.evolving_agent import EvolvingStrategy, RAGEvaluator, RAGEvoAgent
from rdagent.core.exception import CoderError
from rdagent.core.experiment import Experiment
from rdagent.log import rdagent_logger as logger
from rdagent.oai.backend.base import RD_Agent_TIMER_wrapper
class CoSTEER(Developer[Experiment]):
def __init__(
self,
settings: CoSTEERSettings,
eva: RAGEvaluator,
es: EvolvingStrategy,
*args,
evolving_version: int = 2,
with_knowledge: bool = True,
knowledge_self_gen: bool = True,
max_loop: int | None = None,
stop_eval_chain_on_fail: bool = False,
**kwargs,
) -> None:
super().__init__(*args, **kwargs)
self.settings = settings
self.max_loop = settings.max_loop if max_loop is None else max_loop
self.knowledge_base_path = (
Path(settings.knowledge_base_path) if settings.knowledge_base_path is not None else None
)
self.new_knowledge_base_path = (
Path(settings.new_knowledge_base_path) if settings.new_knowledge_base_path is not None else None
)
self.with_knowledge = with_knowledge
self.knowledge_self_gen = knowledge_self_gen
self.evolving_strategy = es
self.evaluator = eva
self.evolving_version = evolving_version
self.stop_eval_chain_on_fail = stop_eval_chain_on_fail
# init rag method
self.rag = (
CoSTEERRAGStrategyV2(
settings=settings,
former_knowledge_base_path=self.knowledge_base_path,
dump_knowledge_base_path=self.new_knowledge_base_path,
evolving_version=self.evolving_version,
)
if self.evolving_version == 2
else CoSTEERRAGStrategyV1(
settings=settings,
former_knowledge_base_path=self.knowledge_base_path,
dump_knowledge_base_path=self.new_knowledge_base_path,
evolving_version=self.evolving_version,
)
)
def get_develop_max_seconds(self) -> int | None:
"""
Get the maximum seconds for the develop task.
Sub classes might override this method to provide a different value.
"""
return None
def _get_last_fb(self) -> CoSTEERMultiFeedback:
fb = self.evolve_agent.evolving_trace[-1].feedback
if fb is None:
raise AssertionError("feedback is None")
if not isinstance(fb, CoSTEERMultiFeedback):
raise TypeError("feedback must be of type CoSTEERMultiFeedback")
return fb
def should_use_new_evo(self, base_fb: CoSTEERMultiFeedback | None, new_fb: CoSTEERMultiFeedback) -> bool:
"""
Compare new feedback with the fallback feedback.
Returns:
bool: True if the new feedback better and False if the new feedback is worse or invalid.
"""
if new_fb is not None and new_fb.is_acceptable():
return True
return False
def develop(self, exp: Experiment) -> Experiment:
# init intermediate items
max_seconds = self.get_develop_max_seconds()
evo_exp = EvolvingItem.from_experiment(exp)
self.evolve_agent = RAGEvoAgent[EvolvingItem](
max_loop=self.max_loop,
evolving_strategy=self.evolving_strategy,
rag=self.rag,
with_knowledge=self.with_knowledge,
knowledge_self_gen=self.knowledge_self_gen,
enable_filelock=self.settings.enable_filelock,
filelock_path=self.settings.filelock_path,
stop_eval_chain_on_fail=self.stop_eval_chain_on_fail,
)
# Evolving the solution
start_datetime = datetime.now()
fallback_evo_exp = None
fallback_evo_fb = None
reached_max_seconds = False
evo_fb = None
iteration_count = 0
# Save initial state before first iteration
self._save_intermediate_results(evo_exp, None, 0, start_datetime)
for evo_exp in self.evolve_agent.multistep_evolve(evo_exp, self.evaluator):
iteration_count += 1
if not isinstance(evo_exp, Experiment):
raise TypeError("evo_exp must be an instance of Experiment")
evo_fb = self._get_last_fb()
update_fallback = self.should_use_new_evo(
base_fb=fallback_evo_fb,
new_fb=evo_fb,
)
if update_fallback:
fallback_evo_exp = deepcopy(evo_exp)
fallback_evo_fb = deepcopy(evo_fb)
fallback_evo_exp.create_ws_ckp() # NOTE: creating checkpoints for saving files in the workspace to prevent inplace mutation.
logger.log_object(evo_exp.sub_workspace_list, tag="evolving code")
for sw in evo_exp.sub_workspace_list:
logger.info(f"evolving workspace: {sw}")
# Save intermediate results after each iteration
self._save_intermediate_results(evo_exp, evo_fb, iteration_count, start_datetime)
if max_seconds is not None and (datetime.now() - start_datetime).total_seconds() > max_seconds:
logger.info(f"Reached max time limit {max_seconds} seconds, stop evolving")
reached_max_seconds = True
break
if RD_Agent_TIMER_wrapper.timer.started and RD_Agent_TIMER_wrapper.timer.is_timeout():
logger.info("Global timer is timeout, stop evolving")
break
try:
# Fallback is required because we might not choose the last acceptable evo to submit.
if fallback_evo_exp is not None:
logger.info("Fallback to the fallback solution.")
evo_exp = fallback_evo_exp
evo_exp.recover_ws_ckp()
evo_fb = fallback_evo_fb
if evo_fb is None:
raise AssertionError("multistep_evolve should run at least once")
evo_exp = self._exp_postprocess_by_feedback(evo_exp, evo_fb)
except CoderError as e:
e.caused_by_timeout = reached_max_seconds
raise e
exp.sub_workspace_list = evo_exp.sub_workspace_list
exp.experiment_workspace = evo_exp.experiment_workspace
return exp
def _save_intermediate_results(self, evo_exp, evo_fb, iteration: int, start_datetime) -> None:
"""
Save intermediate CoSTEER results to results/ directory after each iteration.
This ensures results are visible even if CoSTEER takes a long time
or ultimately fails.
Parameters
----------
evo_exp : EvolvingItem
Current evolving experiment
evo_fb : CoSTEERMultiFeedback
Feedback from the evaluator
iteration : int
Current iteration number
start_datetime : datetime
When the develop process started
"""
import json as _json
import os as _os
from datetime import datetime as _dt
try:
# Go up from rdagent/components/coder/CoSTEER/ to project root (5 levels)
project_root = Path(__file__).parent.parent.parent.parent.parent
# Parallel run isolation: use run-specific directory if PARALLEL_RUN_ID is set
parallel_run_id = _os.getenv("PARALLEL_RUN_ID", "0")
if parallel_run_id != "0":
results_dir = project_root / "results" / "runs" / f"run{parallel_run_id}" / "costeer"
else:
results_dir = project_root / "results" / "runs"
results_dir.mkdir(parents=True, exist_ok=True)
# Build summary
summary = {
"timestamp": _dt.now().isoformat(),
"iteration": iteration,
"elapsed_seconds": (_dt.now() - start_datetime).total_seconds(),
"factors": [],
}
# Extract factor info from sub_workspace_list
if hasattr(evo_exp, "sub_workspace_list") and evo_exp.sub_workspace_list:
for i, sw in enumerate(evo_exp.sub_workspace_list):
factor = {"index": i, "file_count": 0, "code_preview": None}
if hasattr(sw, "file_dict") and sw.file_dict:
factor["file_count"] = len(sw.file_dict)
code = sw.file_dict.get("factor.py", "")
if code:
# First 200 chars as preview
factor["code_preview"] = code[:200]
summary["factors"].append(factor)
# Extract feedback info
if evo_fb is not None:
summary["feedback_count"] = len(evo_fb) if hasattr(evo_fb, "__len__") else 0
accepted = 0
rejected = 0
for fb in evo_fb:
if fb is not None:
if fb.is_acceptable():
accepted += 1
else:
rejected += 1
summary["accepted"] = accepted
summary["rejected"] = rejected
summary["status"] = "accepted" if accepted > 0 else "rejected"
else:
summary["feedback_count"] = 0
summary["accepted"] = 0
summary["rejected"] = 0
summary["status"] = "initialized"
# Write JSON file
ts = _dt.now().strftime("%Y%m%d_%H%M%S")
if parallel_run_id != "0":
json_path = results_dir / f"costeer_run{parallel_run_id}_iter{iteration:02d}_{ts}.json"
else:
json_path = results_dir / f"costeer_iter{iteration:02d}_{ts}.json"
with open(json_path, "w", encoding="utf-8") as f:
_json.dump(summary, f, ensure_ascii=False, indent=2, default=str)
logger.info(
f"CoSTEER iteration {iteration}: "
f"accepted={summary.get('accepted', 0)}, "
f"rejected={summary.get('rejected', 0)}, "
f"saved to {json_path.name}"
)
except Exception as e:
logger.warning(f"Failed to save intermediate CoSTEER results: {e}")
def _exp_postprocess_by_feedback(self, evo: Experiment, feedback: CoSTEERMultiFeedback) -> Experiment:
"""
Responsibility:
- Raise Error if it failed to handle the develop task
-
"""
if not isinstance(evo, Experiment):
raise TypeError("evo must be an instance of Experiment")
if not isinstance(feedback, CoSTEERMultiFeedback):
raise TypeError("feedback must be an instance of CoSTEERMultiFeedback")
if len(evo.sub_workspace_list) != len(feedback):
raise ValueError("Length of sub_workspace_list must match length of feedback")
# FIXME: when whould the feedback be None?
failed_feedbacks = [
f"- feedback{index + 1:02d}:\n - execution: {f.execution}\n - return_checking: {f.return_checking}\n - code: {f.code}"
for index, f in enumerate(feedback)
if f is not None and not f.is_acceptable()
]
if len(failed_feedbacks) == len(feedback):
feedback_summary = "\n".join(failed_feedbacks)
raise CoderError(f"All tasks are failed:\n{feedback_summary}")
return evo