jlfwong/speedscope

Support trailing whitespace & floating point values with Brendan Gregg collapsed stack format

Open

#157 geöffnet am 5. Sept. 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (181 Forks)batch import
feature requesthelp wanted

Repository-Metriken

Stars
 (4.040 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 17h) (1 gemergte PR in 30 T)

Beschreibung

So I have some custom tool that generates data in a output similar to Brendan Gregg's stack format, that works with flamegraph.pl, but is throwing issues with my custom file.

I found there were a few different issues in my file:

  • a trailing newline in the file causing an error
  • my values weren't integers
  • some of my values were negative

Not sure if we could raise a better error message, for those?

Maybe it'd also be useful mentioning that in the docs, I am happy to make a PR, if there is a way to do that in docs?

Contributor Guide

Research-Richtung
Überprüfe die Parsing Logik für Brendan Greggs zusammengeklapptes Stack Format und füge Unterstützung für abschließende Leerzeichen, Gleitkommawerte und negative Werte hinzu. Verbessere auch die Fehlermeldungen für ungültige Eingaben.
Tech Stack
typescript
Domain
toolingperformance
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75