vernier 1.1.0 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa7e8ccc4ac1312ccffb820ec0c6690209725a26ff0f5a9945a508edca073975
-  data.tar.gz: a0f7b10b284dd8fd387a8b12e15a2c9e8535e07f9d19f37c0734b08c93693956
+  metadata.gz: 0e53177832167777357ffdfc79c26f1957a8707804cb85349560faa6d433fd49
+  data.tar.gz: bc89ef70d31f160e8a874c27a2b9bb6479c1a4e631549dd4ac583b7137177660
 SHA512:
-  metadata.gz: 7effed8c46b5110437f12d3fb8a4c4707e334c7edd51d4838198bb8dfb56cdd48c6d1700464be2d32b979f97eafc4c45a2406a4028cceadcac3bd71c1520babb
-  data.tar.gz: 4f11e253e293fd455d1eff820262fb6ea217bf0e3c574dd14bc15c3638e576fd3fb25ee84c3cdeb7151b2877d3f918b4157262f3268797482c192f3935b8768a
+  metadata.gz: 642f607cc4998decb398e88dd196b920ae9592c139fdf7527ad7feb383579b22ff697fa4dede77d62866784415f798d8b020cdb0f8122cb30ea02a6fa66cc7cb
+  data.tar.gz: '03749312aad61f1c336863dece9417afc01df487a6f3f44f7c55e94fcb81b9df32e2aabd232b61a22b7add89f969cfe2a685816ea38cfd66447ac2f7813ed96a'

data/exe/vernier

@@ -1,6 +1,7 @@
 #!/usr/bin/env ruby
 
 require "optparse"
+require "vernier/version"
 
 banner = <<-END
 Usage: vernier run [FLAGS] -- COMMAND
@@ -10,6 +11,8 @@ END
 
 options = {}
 parser = OptionParser.new(banner) do |o|
+  o.version = Vernier::VERSION
+
   o.on('--output [FILENAME]', String, "output filename") do |s|
     options[:output] = s
   end

data/ext/vernier/vernier.cc

@@ -554,19 +554,6 @@ struct StackTable {
         }
     }
 
-    // FIXME: probably should remove
-    void clear() {
-        frame_map.clear();
-        func_map.clear();
-        func_info_list.clear();
-
-        {
-            const std::lock_guard<std::mutex> lock(stack_mutex);
-            stack_node_list.clear();
-            root_stack_node.children.clear();
-        }
-    }
-
     StackNode *convert_stack(StackTable &other, int original_idx) {
         if (original_idx < 0) {
             return &root_stack_node;
@@ -1036,13 +1023,15 @@ class SampleList {
         }
 
         void record_sample(int stack_index, TimeStamp time, Category category) {
-            if (
-                    !empty() &&
-                    stacks.back() == stack_index &&
-                    categories.back() == category)
-            {
-                // We don't compare timestamps for de-duplication
-                weights.back() += 1;
+          // FIXME: probably better to avoid generating -1 higher up.
+          // Currently this happens when we measure an empty stack. Ideally we would have a better representation
+          if (stack_index < 0)
+            return;
+
+          if (!empty() && stacks.back() == stack_index &&
+              categories.back() == category) {
+            // We don't compare timestamps for de-duplication
+            weights.back() += 1;
             } else {
                 stacks.push_back(stack_index);
                 timestamps.push_back(time);

data/lib/vernier/output/firefox.rb

@@ -213,7 +213,6 @@ module Vernier
           @profile = profile
           @categorizer = categorizer
           @tid = tid
-          @allocations = allocations
           @name = name
           @is_main = is_main
           if is_main.nil?
@@ -227,6 +226,15 @@ module Vernier
 
           @samples = samples
 
+          if allocations
+            allocation_samples = allocations[:samples].dup
+            allocation_samples.map! do |sample|
+              @stack_table.convert(profile._stack_table, sample)
+            end
+            allocations = allocations.merge(samples: allocation_samples)
+          end
+          @allocations = allocations
+
           timestamps ||= [0] * samples.size
           @weights, @timestamps = weights, timestamps
           @sample_categories = sample_categories || ([0] * samples.size)

data/lib/vernier/result.rb

@@ -102,7 +102,7 @@ module Vernier
       end
 
       def line
-        result.frame_line_idx(idx)
+        result._stack_table.frame_line_no(idx)
       end
 
       def to_s

data/lib/vernier/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Vernier
-  VERSION = "1.1.0"
+  VERSION = "1.1.2"
 end

data/vernier.gemspec

@@ -12,7 +12,10 @@ Gem::Specification.new do |spec|
   spec.description = "Next-generation Ruby 3.2.1+ sampling profiler. Tracks multiple threads, GVL activity, GC pauses, idle time, and more."
   spec.homepage = "https://github.com/jhawthorn/vernier"
   spec.license = "MIT"
-  spec.required_ruby_version = ">= 3.2.1"
+
+  unless ENV["IGNORE_REQUIRED_RUBY_VERSION"]
+    spec.required_ruby_version = ">= 3.2.1"
+  end
 
   spec.metadata["homepage_uri"] = spec.homepage
   spec.metadata["source_code_uri"] = spec.homepage

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vernier
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.2
 platform: ruby
 authors:
 - John Hawthorn
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-01 00:00:00.000000000 Z
+date: 2024-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -117,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.9
+rubygems_version: 3.5.11
 signing_key:
 specification_version: 4
 summary: A next generation CRuby profiler