Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #272 +/- ##
=======================================
Coverage 91.20% 91.20%
=======================================
Files 19 19
Lines 1592 1592
=======================================
Hits 1452 1452
Misses 140 140 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 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 |
Note
Use this template for bug fixes only. For enhancements/new features, use the feature template and get maintainer approval in an issue/discussion before opening a PR.
Pre-PR Checklist
Summary
Fixes #164.
Bug / Issue
Mesa-LLM is not yet compatible with Mesa 4. The codebase still contains Mesa 4-breaking usage such as
mesa.space,model.steps, and other API assumptions that are still being worked through in separate issues and PRs.Without this fix, users can install Mesa 4 and hit import/runtime failures even though Mesa 4 support is not complete yet.
Implementation
Pin Mesa to
<4for now by changing the dependency from:mesa[rec]>=3.0to:
mesa[rec]>=3.0,<4Additional Notes
This PR does not add Mesa 4 support. It only prevents installation of an unsupported major version while the migration is coordinated separately.