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

Only run benchmark bot on actual code changes #1999

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

EwoutH
Copy link
Contributor

@EwoutH EwoutH commented Jan 24, 2024

Only run if either an Python file has changed or if the benchmark workflow itself is changed.

It can be more fine-grained, but this should catch most docs-only cases.

Only run if either an Python file has changed or if the benchmark workflow itself is changed.

It can be more fine-grained, but this should catch most docs-only cases.
@EwoutH EwoutH added ci Release notes label ignore-for-release PRs that aren't included in the release notes labels Jan 24, 2024
@EwoutH EwoutH requested review from rht and Corvince January 24, 2024 12:54
Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
Schelling small 🔵 -0.6% [-0.8%, -0.4%] 🔵 -0.1% [-0.4%, +0.1%]
Schelling large 🔵 -1.2% [-1.6%, -0.8%] 🔵 -1.7% [-2.8%, -0.4%]
WolfSheep small 🔵 -2.2% [-2.5%, -1.9%] 🔵 -0.7% [-0.9%, -0.6%]
WolfSheep large 🔵 -3.5% [-7.1%, -0.7%] 🔵 -2.0% [-3.3%, -0.8%]
BoidFlockers small 🔵 +2.2% [+1.9%, +2.5%] 🔵 +0.8% [-0.0%, +1.7%]
BoidFlockers large 🔵 +2.6% [+1.9%, +3.3%] 🔵 +0.1% [-0.2%, +0.3%]

@rht rht merged commit 3e73e9b into projectmesa:main Jan 24, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Release notes label ignore-for-release PRs that aren't included in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants