jlfwong/speedscope

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

Ouverte

#157 ouverte le 5 sept. 2018

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)TypeScript (319 forks)batch import
feature requesthelp wanted

Métriques du dépôt

Stars
 (6 727 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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?

Guide contributeur

Direction de recherche
Inspectez la logique d'analyse pour le format de pile pliée de Brendan Gregg et ajoutez la prise en charge des espaces de fin, des valeurs à virgule flottante et des valeurs négatives. Améliorez également les messages d'erreur pour les entrées invalides.
Stack technique
typescript
Domaine
toolingperformance
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75