fix: vision radius bug and improved LLM resilience#95
fix: vision radius bug and improved LLM resilience#95uday-codes69 wants to merge 3 commits intomesa:mainfrom
Conversation
…key error messages
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
- Fixed hardcoded vision radius in LLMAgent._build_observation - Fixed missing neighbor detection logic for ContinuousSpace in LLMAgent - Updated ModuleLLM return type hints to match litellm's ModelResponse - Added InternalServerError and ServiceUnavailableError to retryable exceptions
for more information, see https://pre-commit.ci
|
Hey @uday-codes69 , hope you are doing well :) Correct me if I'm wrong but I think PR #61 is already fixing the vision radius bug mentioned here. I may be wrong could you please go through it and confirm ? Thanks:) |
mesa_llm/llm_agent.py:
Fixed a bug where the vision radius was hardcoded to 1 in grid-based environments, ignoring the agent's vision parameter.
Fixed a NameError in ContinuousSpace logic by implementing the correct neighbor detection method.
mesa_llm/module_llm.py:
Enhanced reliability by adding InternalServerError and ServiceUnavailableError to RETRYABLE_EXCEPTIONS.
Corrected type hints and docstrings for generate/agenerate
to return Any (standardizing on litellm's ModelResponse object).