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 benchmarks.yml #2043

Merged
merged 1 commit into from
Feb 19, 2024
Merged

Update benchmarks.yml #2043

merged 1 commit into from
Feb 19, 2024

Conversation

Corvince
Copy link
Contributor

This should fix benchmarks not working for the experimental grid cell PR

Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
Schelling small 🔵 -0.1% [-0.3%, +0.2%] 🔵 -0.4% [-0.6%, -0.3%]
Schelling large 🔵 -1.4% [-2.9%, -0.1%] 🔵 -0.7% [-1.9%, +0.4%]
WolfSheep small 🔵 -0.5% [-0.8%, -0.1%] 🔵 -0.2% [-0.3%, -0.0%]
WolfSheep large 🔵 -1.5% [-2.3%, -0.8%] 🔵 -5.7% [-8.2%, -2.8%]
BoidFlockers small 🔵 -1.0% [-1.5%, -0.6%] 🔵 +0.5% [-0.1%, +1.1%]
BoidFlockers large 🔵 -0.8% [-1.5%, -0.2%] 🔵 +1.8% [+1.2%, +2.4%]

@@ -29,7 +29,7 @@ jobs:
- name: Add project directory to PYTHONPATH
run: echo "PYTHONPATH=$PYTHONPATH:$(pwd)" >> $GITHUB_ENV
- name: Install dependencies
run: pip install numpy pandas tqdm tabulate
run: pip install numpy pandas tqdm tabulate matplotlib solara networkx
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was about to ask if the Solara import could be made optional, but at this point it shouldn't matter because uv will turn pip install fast.

@rht rht merged commit 8418b57 into main Feb 19, 2024
11 checks passed
@rht rht deleted the Corvince-patch-1 branch February 19, 2024 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants