Skip to content

Rust graph (Peter) noop#176

Closed
Peter554 wants to merge 6 commits intopython-grimp:masterfrom
Peter554:peter.rust-graph-noop
Closed

Rust graph (Peter) noop#176
Peter554 wants to merge 6 commits intopython-grimp:masterfrom
Peter554:peter.rust-graph-noop

Conversation

@Peter554
Copy link
Collaborator

No description provided.

seddonym and others added 6 commits January 15, 2025 16:46
This is the main graph implemented in Rust - but it isn't plumbed into
Python yet.
This adds a wrapper around the Rust-graph in lib.rs which handles the
interface between Python and Rust. At the same time, it changes the
Python ImportGraph so it uses Rust instead.
This is faster, but not cryptographically safe. That shouldn't matter for our
purposes.
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 17, 2025

CodSpeed Performance Report

Merging #176 will not alter performance

Comparing Peter554:peter.rust-graph-noop (9a2edbd) with master (957ff81)

Summary

✅ 1 untouched benchmarks
⁉️ 3 dropped benchmarks

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

Benchmarks breakdown

Benchmark master Peter554:peter.rust-graph-noop Change
⁉️ test_build_django_from_cache 224.4 ms N/A N/A
⁉️ test_deep_layers_large_graph 1.3 s N/A N/A
⁉️ test_top_level_large_graph 260.8 ms N/A N/A

@Peter554 Peter554 closed this Jan 17, 2025
@Peter554 Peter554 deleted the peter.rust-graph-noop branch February 7, 2025 08:32
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