facebookarchive/prepack

Make Prepack heap graph visualization to be consistently with different serialization options

Open

#1 309 ouverte le 2 janv. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (520 forks)batch import
bughelp wantedwebsite

Métriques du dépôt

Stars
 (14 268 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Currently, Prepack heap graph visualization is based on the default Prepack mode so it may diverge from the real generated code when options are applied. For example, when --inlineExpressions is applied, many objects are inlined into its parent object so there is no identifier generated for it while heap visualization still shows their identifiers which is confusing.(Not sure what to show though).

Here are some options that may change the graph but not well tested:

  1. --inlineExpressions.
  2. --debugNames
  3. --additionalFunctions.
  4. --react

For this issue, we should a least fix for "--inlineExpressions" since it is default on website. Also, we should make heap visualization feature is disabled when "--additionalFunctions/--react" is specified since we haven't tested it yet.

Guide contributeur

Direction de recherche
Examinez comment la visualisation du graphe de tas gère actuellement les options de sérialisation comme inlineExpressions, debugNames, additionalFunctions et react. Concentrez vous sur la garantie de cohérence pour inlineExpressions car il est par défaut sur le site Web. Envisagez de désactiver la visualisation pour les options non testées.
Stack technique
javascript
Domaine
backendtooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60