Skip to content
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

Update Ruff to 0.3.4; apply ruff format . #2088

Merged
merged 2 commits into from
Mar 25, 2024
Merged

Conversation

rht
Copy link
Contributor

@rht rht commented Mar 24, 2024

No description provided.

Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
Schelling small 🔵 -1.1% [-1.3%, -0.8%] 🔵 -0.7% [-0.9%, -0.5%]
Schelling large 🔵 +1.5% [+0.3%, +2.5%] 🔵 +3.1% [-3.8%, +9.6%]
WolfSheep small 🔵 +1.3% [+1.1%, +1.5%] 🔵 +0.6% [+0.5%, +0.8%]
WolfSheep large 🔵 +3.1% [+2.4%, +3.9%] 🔴 +15.0% [+11.5%, +18.3%]
BoidFlockers small 🔵 +1.7% [+1.0%, +2.4%] 🔵 -0.8% [-1.6%, +0.1%]
BoidFlockers large 🔵 +1.4% [+0.7%, +2.1%] 🔵 -0.3% [-0.9%, +0.3%]

@rht rht merged commit 51b4e9a into projectmesa:main Mar 25, 2024
11 of 12 checks passed
@rht rht deleted the fix_test branch March 25, 2024 19:05
@EwoutH EwoutH added maintenance Release notes label ignore-for-release PRs that aren't included in the release notes labels Apr 18, 2024
vitorfrois pushed a commit to vitorfrois/mesa that referenced this pull request Jul 15, 2024
* Update Ruff to 0.3.4; apply ruff format .

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release PRs that aren't included in the release notes maintenance Release notes label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants