rubyatscale / singed Goto Github PK

View Code? Open in Web Editor NEW

384.0 384.0 9.0 60 KB

Get a flamegraph anywhere in your code base. Powered by stackprof, rbspy, and speedscope

License: MIT License

Ruby 99.35% Shell 0.65%

flamegraphs rails ruby

singed's People

Contributors

Stargazers

Watchers

Forkers

aesthetikx overload119 cxz jean-francois-labbe berkos iq-scm viamin joeljunstrom danielgilchrist

singed's Issues

Running in project with rspec

It needs to clarify at README that in projects with rspec, the gem 'singed' needs to be at same group as rspecgem in your Gemfile.

Otherwise you get this:

bundle exec singed -- bin/rails runner 'User.all.to_a'
You can remove `require 'dalli/cas/client'` as this code has been rolled into the standard 'dalli/client'.
bundler: failed to load command: singed (/Users/kivanio/.rbenv/versions/3.2.1/bin/singed)
/Users/kivanio/.rbenv/versions/3.2.1/lib/ruby/gems/3.2.0/gems/singed-0.1.1/lib/singed/rspec.rb:3:in `<main>': undefined method `configure' for RSpec:Module (NoMethodError)

RSpec.configure do |config|
     ^^^^^^^^^^

Received NoMethodError: undefined method `print_json'

(this is probably an edge case)

My project had stackprof on 0.2.12 in the Gemfile.lock. It turns out print_json is not defined in this version. It might help someone in the future if you update your gemspec to ensure a supported version of stackprof is installed:

spec.add_dependency 'stackprof', '>= 0.2.13'

Automate releasing

Right now, it's me manually pulling and running rake release. Ideally, this happens automatically on main when the gemspec version changes, or if it is manually run.

Tried this, but without luck so far:

It is currently failing with:

singed 0.2.2 built to pkg/singed-0.2.2.gem.
rake aborted!
There are files that need to be committed first.
/home/runner/work/singed/singed/vendor/bundle/ruby/3.1.0/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
/opt/hostedtoolcache/Ruby/3.1.4/x64/bin/bundle:25:in `load'
/opt/hostedtoolcache/Ruby/3.1.4/x64/bin/bundle:25:in `<main>'
Tasks: TOP => release => release:guard_clean
(See full trace by running task with --trace)

I thought it could be the cache step, but I've added vendor/bundler to .gitignore . It's unfortunate it doesn't tell you what is uncommitted.

My next thought is to add a git status to the build, but that might not trigger in the right place. If that doesn't show it, then I think it's a matter of adding rake tasks to do git status before various steps 🤔

`label` cannot be provided as a keyword

The readme mentions:

If you are calling it in a loop, or with different variations, you can include a label on the filename:

flamegraph(label: "rspec") {
  # your code here
}

This is not accurate, as it results in:

     ArgumentError:
       unknown keyword: :label

If the library wants to support both a normal parameter and a keyword arg, it could use:

def flamegraph(_label = nil, label: _label, open: true, ignore_gc: false, interval: 1000, &block)

Add License to gemspec

License scanners indicate this gem license is "Unknown" since it's not in the gemspec.

Gem::Specification.new do |spec|
  spec.name          = 'singed'

  spec.version       = '0.2.0'
  spec.authors       = ['Josh Nichols']
  spec.email         = ['[email protected]']
+ spec.licenses = 'MIT'

Add `start` and `stop` methods for when code isn't conveniently block wrappable

stackprof supports using start / stop in addition to the block usage. This is helpful for when things aren't cleanly block wrapped.

cc @baweaver who requested it

Need to create folder tmp/speedscope before using CLI

I needed to run mkdir tmp/speedscope to make it work

bundle exec singed -- bin/rails runner 'User.all.to_a'
You can remove `require 'dalli/cas/client'` as this code has been rolled into the standard 'dalli/client'.
🔥📈 Singed needs to run as root, but will drop permissions back to your user. Prompting with sudo now...
Password:
$ sudo --preserve-env rbspy record --format speedscope --file tmp/speedscope/speedscope-cli-20230309144106.json --silent -- bin/rails runner User.all.to_a
Dropping permissions: running Ruby command as user kivanio
rbspy is recording traces. Press Ctrl+C to stop.
You can remove `require 'dalli/cas/client'` as this code has been rolled into the standard 'dalli/client'.
Wrote raw data to /Users/kivanio/Library/Caches/rbspy/2023-03-09-lxTiyfen4D.raw.gz
Wrote formatted output to tmp/speedscope/speedscope-cli-20230309144106.json
Something went wrong while rbspy was sampling the process. Here's what we know:
- Failed to create output file tmp/speedscope/speedscope-cli-20230309144106.json
- No such file or directory (os error 2)
bundler: failed to load command: singed (/Users/kivanio/.rbenv/versions/3.2.1/bin/singed)
/Users/kivanio/.rbenv/versions/3.2.1/lib/ruby/gems/3.2.0/gems/singed-0.1.1/lib/singed/cli.rb:147:in `system': Command failed with exit 1: sudo (RuntimeError)

rubyatscale / singed Goto Github PK

singed's People

Contributors

Stargazers

Watchers

Forkers

singed's Issues

Running in project with rspec

Received NoMethodError: undefined method `print_json'

Automate releasing

`label` cannot be provided as a keyword

Add License to gemspec

Add `start` and `stop` methods for when code isn't conveniently block wrappable

Need to create folder tmp/speedscope before using CLI

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent