facebookarchive/prepack

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

Open

#1309 aperta il 2 gen 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
bughelp wantedwebsite

Metriche repository

Star
 (14.268 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Investigare come la visualizzazione del grafico heap gestisce attualmente le opzioni di serializzazione come inlineExpressions, debugNames, additionalFunctions e react. Concentrarsi sul garantire la coerenza per inlineExpressions poiché è predefinito sul sito web. Considerare di disabilitare la visualizzazione per le opzioni non testate.
Tech stack
javascript
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60