-
Notifications
You must be signed in to change notification settings - Fork 880
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: Remove place_agent duplicate warnings #2086
Conversation
df54bbe
to
8267299
Compare
Performance benchmarks:
|
Can you split of the ruff update in a separate PR? I like to keep PRs atomic. |
Done. |
Thanks! I guess it isn’t allowed in current Mesa to initialize agents with a Also, which warning is this removing exactly? |
Performance benchmarks:
|
Allowed but not recommended.
|
Because of #2083, some faulty test code have been caught. cc: @puer-robustus