atom/teletype

Long activation time

Open

#201 ouverte le 15 nov. 2017

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (2 400 stars) (416 forks)batch import
help wanted

Description

Prerequisites

  • Put an X between the brackets on this line if you have done all of the following:

Description

Teletype is taking a long time to activate for me at around ~190ms. Looks like deferred activation isn't possible, but would it be feasible to defer some requires until they're needed? For example, until you try to host or join a portal, are the majority of requires needed?

Steps to Reproduce

  1. Enable teletype
  2. Reload atom
  3. Look at how long teletype took to activate

Expected behavior: Preferably <25ms

Actual behavior: ~190ms

Reproduces how often: 100%

Versions

Atom    : 1.24.0-dev-192cde85a
Electron: 1.6.15
Chrome  : 56.0.2924.87
Node    : 7.4.0

Windows 10 x64.

Guide contributeur

Stack technique
javascript
Domaine
performance
Type d'issue
performance
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.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Atom package developmentJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
The issue suggests deferring requires until needed. Look at the main entry point (e.g., index.js or lib/teletype.js) to identify all require statements. Consider lazy loading modules that are only needed for hosting or joining portals. Profile the activation time using Atom's timecop feature to measure improvement. Check if any side effects occur from deferred loading.