Skip to content

Very slow compilation in beta.11 version #15644

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Yomyer opened this issue Sep 18, 2018 · 8 comments
Closed

Very slow compilation in beta.11 version #15644

Yomyer opened this issue Sep 18, 2018 · 8 comments
Labels

Comments

@Yomyer
Copy link
Contributor

Yomyer commented Sep 18, 2018

Bug Report

Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.

Ionic:

   ionic (Ionic CLI)          : 4.1.2 (/usr/local/lib/node_modules/ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.10
   @angular-devkit/core       : 0.7.5
   @angular-devkit/schematics : 0.7.5
   @angular/cli               : 6.1.2
   @ionic/ng-toolkit          : 1.0.7
   @ionic/schematics-angular  : 1.0.5

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : browser 5.0.4, ios 4.5.4
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.0.5, cordova-plugin-ionic-webview 1.1.1, (and 14 other plugins)

System:

   ios-deploy : 1.9.2
   ios-sim    : 6.1.2
   NodeJS     : v8.11.1 (/usr/local/Cellar/node@8/8.11.1/bin/node)
   npm        : 5.6.0
   OS         : macOS High Sierra
   Xcode      : Xcode 9.4.1 Build version 9F2000

Describe the Bug
It is very slow to compile, with version 7 it went much faster it becomes impossible to debug, as it takes about 30 seconds to refresh.

Steps to Reproduce
Steps to reproduce the behavior:

  1. Use this command ionic cordova run browser -l or ionic serve
  2. It is very slow to compile, with version 7 it went much faster it becomes impossible to debug, as it takes about 30 seconds to refresh.
@ionitron-bot ionitron-bot bot added the triage label Sep 18, 2018
@PhilippeSchraepen
Copy link

PhilippeSchraepen commented Sep 18, 2018

I'm also having this problem. I've rolled the versions back one-by-one and it seems that the problem was introduced in 4.0.0-beta.9

@mhartington
Copy link
Contributor

So 2 things:

  1. Is there a sample project to inspect? Using a starter project from the CLI, Im seeing about 5second rebuild times, which is pretty average across projects.
[ng] ℹ 「wdm」: Compiled successfully.
[ng] ℹ 「wdm」: Compiling...
[ng] Date: 2018-09-18T14:34:06.072Z - Hash: 5efda5e90e64deebd59c - Time: 5642ms
[ng] 174 unchanged chunks
[ng] ℹ 「wdm」: Compiled successfully.
[ng] ℹ 「wdm」: Compiling...
[ng] Date: 2018-09-18T14:34:20.177Z - Hash: 5efda5e90e64deebd59c - Time: 5627ms
[ng] 174 unchanged chunks
[ng] ℹ 「wdm」: Compiled successfully.
  1. Since this is which compilation, it's more on the Angular CLI side of things, since it is responsible for all the building in V4 project.

@mhartington mhartington added needs: reply the issue needs a response from the user and removed triage labels Sep 18, 2018
@PhilippeSchraepen
Copy link

PhilippeSchraepen commented Sep 18, 2018

So 2 things:

  1. Is there a sample project to inspect? Using a starter project from the CLI, Im seeing about 5second rebuild times, which is pretty average across projects.
[ng] ℹ 「wdm」: Compiled successfully.
[ng] ℹ 「wdm」: Compiling...
[ng] Date: 2018-09-18T14:34:06.072Z - Hash: 5efda5e90e64deebd59c - Time: 5642ms
[ng] 174 unchanged chunks
[ng] ℹ 「wdm」: Compiled successfully.
[ng] ℹ 「wdm」: Compiling...
[ng] Date: 2018-09-18T14:34:20.177Z - Hash: 5efda5e90e64deebd59c - Time: 5627ms
[ng] 174 unchanged chunks
[ng] ℹ 「wdm」: Compiled successfully.
  1. Since this is which compilation, it's more on the Angular CLI side of things, since it is responsible for all the building in V4 project.

I've upgraded an existing (private) project, so maybe the problem doens't occur when generating a new project. I've also upgraded ionic-cli and all other dependencies, without any result.

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Sep 18, 2018
@mhartington
Copy link
Contributor

Is there a sample project to inspect?

Still would like a project to inspect and debug. As compilation is still done with the Angular-CLI, I dont think there's anything on our end that is responsible for compilation times, but I would like to test a project. Otherwise, I'll need to close this issue.

@mhartington mhartington added needs: reply the issue needs a response from the user and removed triage labels Sep 18, 2018
@godenji
Copy link

godenji commented Sep 18, 2018

What solved it for me was upgrading to Node 10.10.0 and NPM 6.4.1, along with removing existing node deps and reinstalling.

Previously beta 11 was horrendously slow, but after applying above steps rebuild times are quite fast (3-4 seconds), faster than ever in fact.

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Sep 18, 2018
@mhartington mhartington reopened this Sep 18, 2018
@PhilippeSchraepen
Copy link

What solved it for me was upgrading to Node 10.10.0 and NPM 6.4.1, along with removing existing node deps and reinstalling.

Previously beta 11 was horrendously slow, but after applying above steps rebuild times are quite fast (3-4 seconds), faster than ever in fact.

This solved it for me too!

@netsesame2
Copy link

Update to Node 10 is right. Compiling time from 79s to 7s.

@ionitron-bot
Copy link

ionitron-bot bot commented Nov 1, 2018

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Nov 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants