feat: implement quadtree

.github/workflows/build.yml

@@ -94,6 +94,6 @@ jobs:
           components: clippy
 
       - name: Clippy check
-        run: cargo clippy --workspace --tests --examples --all-features
+        run: cargo clippy --workspace --tests --examples --all-features -- -D warnings
 
 

Cargo.toml

@@ -6,9 +6,10 @@ resolver = "2"
 
 members = [
     "server",
+    "quadtree",
     "chunk",
     "generator-build",
-    "generator"
+    "generator",
 ]
 
 [profile.dev]
@@ -18,19 +19,15 @@ opt-level = 1
 opt-level = 3
 
 [profile.release]
-#debug = true
 lto = "fat"
 codegen-units = 1
-#strip = false
-#panic = "abort"
 
 [profile.bench]
-#debug = true
 lto = "fat"
 codegen-units = 1
-#panic = "abort"
 
 [workspace.dependencies]
 chunk = { path = "chunk" }
 generator-build = { path = "generator-build" }
 generator = { path = "generator" }
+quadtree = { path = "quadtree" }

deny.toml

@@ -219,7 +219,8 @@ skip = [
     "event-listener",
     "base64",
     "regex-syntax",
-    "regex-automata"
+    "regex-automata",
+    "glam"
     #{ crate = "[email protected]", reason = "you can specify a reason why it can't be updated/removed" },
 ]
 # Similarly to `skip` allows you to skip certain crates during duplicate

generator-build/Cargo.toml

@@ -10,11 +10,11 @@ publish = false
 anyhow = "1.0.81"
 heck = "0.5.0"
 itertools = "0.12.1"
-prettyplease = "0.2.16"
+prettyplease = "0.2.17"
 proc-macro2 = "1.0.79"
 quote = "1.0.35"
 serde = { version = "1.0.197", features = ["derive"] }
-serde_json = "1.0.114"
+serde_json = "1.0.115"
 syn = "2.0.55"
 
 [lints.rust]

quadtree/.gitignore

@@ -0,0 +1 @@
+/target

quadtree/Cargo.toml

@@ -0,0 +1,31 @@
+[package]
+name = "quadtree"
+version = "0.1.0"
+edition = "2021"
+authors = ["Andrew Gazelka <[email protected]>"]
+readme = "README.md"
+publish = false
+
+[dependencies]
+glam = "0.27.0"
+itertools = "0.12.1"
+num-traits = "0.2.18"
+
+[dev-dependencies]
+divan = "0.1.14"
+rand = "0.8.5"
+
+[[bench]]
+name = "rebuild"
+harness = false
+
+[lints.clippy]
+complexity = "deny"
+
+nursery = { level = "deny", priority = -1 }
+redundant_pub_crate = "allow"
+
+pedantic = "deny"
+perf = "deny"
+style = "deny"
+suspicious = "deny"

quadtree/benches/rebuild.rs

@@ -0,0 +1,121 @@
+use divan::{AllocProfiler, Bencher};
+use quadtree::rebuild::MoveElement;
+use rand::prelude::SliceRandom;
+
+#[global_allocator]
+static ALLOC: AllocProfiler = AllocProfiler::system();
+
+fn main() {
+    divan::main();
+}
+
+// Register a `fibonacci` function and benchmark it over multiple cases.
+
+//
+const LENS: &[usize] = &[64, 512, 4096, 32768, 262_144];
+
+#[divan::bench(
+    args = LENS,
+)]
+fn rebuild_single(bencher: Bencher, len: usize) {
+    let v = (0..len).collect::<Vec<_>>();
+
+    let changes = vec![quadtree::rebuild::MoveElement {
+        remove_from_idx: 1,
+        insert_to_idx: 3,
+    }];
+
+    bencher.bench(|| {
+        quadtree::rebuild::apply_vec(&v, &changes, &mut [0, 1, 2, 3, 4]);
+    });
+}
+
+#[divan::bench(
+args = LENS,
+)]
+fn rebuild_every(bencher: Bencher, len: usize) {
+    let v = (0..len).collect::<Vec<_>>();
+
+    let arr = 0..len;
+
+    // shuffle the array
+    let mut arr = arr.collect::<Vec<_>>();
+    arr.shuffle(&mut rand::thread_rng());
+
+    let changes = arr
+        .iter()
+        .enumerate()
+        .map(|(i, &x)| quadtree::rebuild::MoveElement {
+            remove_from_idx: i,
+            insert_to_idx: x,
+        })
+        .collect::<Vec<_>>();
+
+    bencher.counter(len).bench(|| {
+        quadtree::rebuild::apply_vec(&v, &changes, &mut [0, 1, 2, 3, 4]);
+    });
+}
+
+#[divan::bench(
+args = LENS,
+)]
+fn rebuild_262144(bencher: Bencher, len: usize) {
+    const LEN: usize = 262_144;
+    let v = (0..LEN).collect::<Vec<_>>();
+
+    let arr = 0..len;
+
+    // shuffle the array
+    let mut arr = arr.collect::<Vec<_>>();
+    arr.shuffle(&mut rand::thread_rng());
+
+    let changes = arr
+        .iter()
+        .enumerate()
+        .map(|(i, &x)| quadtree::rebuild::MoveElement {
+            remove_from_idx: i,
+            insert_to_idx: x,
+        })
+        .collect::<Vec<_>>();
+
+    bencher.bench(|| {
+        quadtree::rebuild::apply_vec(&v, &changes, &mut [0, 1, 2, 3, 4]);
+    });
+}
+
+#[divan::bench(
+args = [64, 512, 4096],
+)]
+fn rebuild_262144_naive(bencher: Bencher, len: usize) {
+    const LEN: usize = 262_144;
+    let v = (0..LEN).collect::<Vec<_>>();
+
+    let arr = 0..len;
+
+    // shuffle the array
+    let mut arr = arr.collect::<Vec<_>>();
+    arr.shuffle(&mut rand::thread_rng());
+
+    let changes = arr
+        .iter()
+        .enumerate()
+        .map(|(i, &x)| quadtree::rebuild::MoveElement {
+            remove_from_idx: i,
+            insert_to_idx: x,
+        })
+        .collect::<Vec<_>>();
+
+    bencher.bench(|| {
+        let mut v = v.clone();
+
+        for &MoveElement {
+            remove_from_idx,
+            insert_to_idx,
+        } in &changes
+        {
+            let elem = v[remove_from_idx];
+            v.remove(remove_from_idx);
+            v.insert(insert_to_idx, elem);
+        }
+    });
+}

quadtree/rustfmt.toml

@@ -0,0 +1,21 @@
+combine_control_expr = true
+comment_width = 100 # https://lkml.org/lkml/2020/5/29/1038
+condense_wildcard_suffixes = true
+control_brace_style = "AlwaysSameLine"
+edition = "2021"
+format_code_in_doc_comments = true
+format_macro_bodies = true
+format_macro_matchers = true
+format_strings = true
+group_imports = "StdExternalCrate"
+imports_granularity = "Crate"
+merge_derives = false
+newline_style = "Unix"
+normalize_comments = true
+normalize_doc_attributes = true
+overflow_delimited_expr = true
+reorder_impl_items = true
+reorder_imports = true
+unstable_features = true
+wrap_comments = true
+