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

ENH: Add counting triangles example #781

Merged
merged 1 commit into from
Sep 30, 2024
Merged

ENH: Add counting triangles example #781

merged 1 commit into from
Sep 30, 2024

Conversation

mtsokol
Copy link
Collaborator

@mtsokol mtsokol commented Sep 27, 2024

Hi @hameerabbasi,

Here's a short example that runs counting triangles with Finch, SciPy, and NetworkX. Two comments:

@mtsokol mtsokol added the enhancement Indicates new feature requests label Sep 27, 2024
@mtsokol mtsokol self-assigned this Sep 27, 2024
Copy link

codspeed-hq bot commented Sep 27, 2024

CodSpeed Performance Report

Merging #781 will degrade performances by 25.26%

Comparing triangles-example (733f44b) with main (a60f0ad)

Summary

❌ 7 regressions
✅ 333 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main triangles-example Change
test_index_slice[side=100-rank=2-format='gcxs'] 2.4 ms 3.2 ms -25.26%
test_elemwise[f=<built-in function add>-backend='Finch'-side=1000] 1.5 ms 1.7 ms -11.83%
test_elemwise[f=<built-in function add>-backend='Finch'-side=100] 728.4 µs 925.2 µs -21.27%
test_elemwise[f=<built-in function add>-backend='Finch'-side=500] 893.8 µs 1,094 µs -18.3%
test_elemwise[f=<built-in function mul>-backend='Finch'-side=1000] 750.1 µs 958.8 µs -21.76%
test_elemwise[f=<built-in function mul>-backend='Finch'-side=100] 706.6 µs 908.3 µs -22.21%
test_elemwise[f=<built-in function mul>-backend='Finch'-side=500] 723.3 µs 935.9 µs -22.72%

@hameerabbasi
Copy link
Collaborator

I see that examples in examples/ directory aren't executed in the CI job anymore, is it intended? I think they should still be executed.

I believe that benchmarks should be moved to CodSpeed benchmarks, see #772, but examples with small datasets should be executed without measuring timings.

@mtsokol
Copy link
Collaborator Author

mtsokol commented Sep 30, 2024

Ok, then I guess this PR is ready anyway.

@mtsokol mtsokol merged commit fbdd5c7 into main Sep 30, 2024
16 of 17 checks passed
@mtsokol mtsokol deleted the triangles-example branch September 30, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants