wip

Author: gurgeous

.gitignore

@@ -1,6 +1,6 @@
-.tool-versions
-*.gem
-Gemfile.lock
-lib/*.so
-lib/kdtree.bundle
-tmp
+*.bundle
+*.so
+/.tool-versions
+/Gemfile.lock
+/pkg/
+/tmp/

README.md

@@ -54,17 +54,17 @@ kd2 = File.open("treefile") { |f| Kdtree.new(f) }
 
 ### Performance
 
-Kdtree is fast. How fast? Using a tree with 1 million points on my i5 2.8ghz:
+Kdtree is fast. How fast? Using a tree with 1 million points on my M1:
 
 ```
-build (init)        3.52s
-nearest point       0.000003s
-nearest 5 points    0.000004s
-nearest 50 points   0.000014s
-nearest 255 points  0.000063s
-
-persist             0.301963s
-read (init)         0.432676s
+build (init)        0.96s
+persist             0.000814s
+read (init)         0.009236s
+
+nearest point       0.000002s
+nearest 5 points    0.000002s
+nearest 50 points   0.000006s
+nearest 255 points  0.000026s
 ```
 
 ### Limitations
@@ -85,7 +85,7 @@ Since this gem was originally released, several folks have contributed important
 
 ### Changelog
 
-Note: This gem is stable, maintained and continues to work great with all modern versions of Ruby MRI. Our CI tests through Ruby 2.7. No need for new releases until something breaks!
+Note: This gem is stable, maintained and continues to work great with all modern versions of Ruby MRI. Our CI tests through Ruby 3.4. No need for new releases until something breaks!
 
 #### 0.5 - May 2025
 

Rakefile

@@ -1,22 +1,12 @@
-require "bundler/setup"
+require "bundler/gem_tasks"
 require "rake/extensiontask"
 require "rake/testtask"
-require "bundler"
-Bundler::GemHelper.install_tasks
-
-#
-# rake-compiler
-#
 
 Rake::ExtensionTask.new("kdtree")
-
-#
-# testing
-#
+task test: :compile
 
 Rake::TestTask.new do
   _1.libs << "test"
   _1.test_files = FileList["test/**/test_*.rb"]
 end
-task test: :compile
 task default: :test

justfile

@@ -1,29 +1,49 @@
 default: test
 
+#
+# ci/test
+#
+
 # check repo - lint & test
 check: lint test
 
 # for ci. don't bother linting on windows
 ci:
   @just test
 
-# format with rubocop
-format: (lint "-a")
+# run tests
+test *ARGS:
+  @just _banner rake test {{ARGS}}
+  @bundle exec rake test {{ARGS}}
+
+#
+# build/release
+#
+
+clean:
+  rm -rf pkg tmp lib/*.bundle lib/*.so
+
+gem-build: check clean
+  @just _banner rake build...
+  @bundle exec rake build
 
 # this will tag, build and push to rubygems
-gem-push: check
+gem-push: check clean
   @just _banner rake release...
   rake release
 
+#
+# lint
+#
+
+# format with rubocop
+format: (lint "-a")
+
 # lint with rubocop
 lint *ARGS:
   @just _banner lint...
   bundle exec rubocop {{ARGS}}
 
-# run tests
-test *ARGS:
-  @just _banner rake test {{ARGS}}
-  @bundle exec rake test {{ARGS}}
 
 #
 # util

kdtree.gemspec

@@ -19,7 +19,15 @@ Gem::Specification.new do |s|
     "source_code_uri" => s.homepage,
   }
 
-  s.files = `git ls-files`.split("\n")
+  s.files = %w[
+    ext/kdtree/extconf.rb
+    ext/kdtree/kdtree.c
+    kdtree.gemspec
+    lib/kdtree.rb
+    LICENSE
+    README.md
+  ]
+
   s.extensions = ["ext/kdtree/extconf.rb"]
   s.require_paths = ["lib"]
 end

test/test_kdtree.rb

@@ -96,6 +96,11 @@ def dont_test_speed
     sizes.each do |s|
       points = (0...s).map { |i| [rand_coord, rand_coord, i] }
 
+      puts
+      puts "x" * 72
+      puts "x with #{s} points"
+      puts "x" * 72
+
       # build
       Benchmark.bm(17) do |bm|
         kdtree = nil
@@ -121,7 +126,6 @@ def dont_test_speed
           end
         end
       end
-      puts
     end
   end