Skip to content
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

0.8.2 is extremely slow #33

Open
snakamura opened this issue Oct 11, 2017 · 5 comments
Open

0.8.2 is extremely slow #33

snakamura opened this issue Oct 11, 2017 · 5 comments

Comments

@snakamura
Copy link

snakamura commented Oct 11, 2017

It became extremely slow when I updated babel-plugin-angularjs-annotate from 0.8.1 to 0.8.2.

time babel --presets es2015 --plugins angularjs-annotate app.es6.js > /dev/null

app.es6.js is around 750KB/25,000 lines.

With 0.8.1

real 0m5.835s
user 0m6.075s
sys 0m0.275s

With 0.8.2

real 2m5.533s
user 2m4.816s
sys 0m0.986s

I'm running babel 6.26.0 with node 6.11.4 on Mac OS 10.12.6.

@schmod
Copy link
Owner

schmod commented Oct 11, 2017

Yikes. I'll take a look.

Feel free to use 0.8.1 if #29 didn't affect you.

@SenHengDesamis
Copy link

We have similar issues too. Transpilation times went up 30s.

0.8.2

desamis-no-MacBook-Air:cc senheng$ yarn add -D babel-plugin-angularjs-annotate
...
success Saved 1 new dependency.
└─ [email protected]
...
[11:39:53] Starting 'js'...
[11:40:41] Finished 'js' after 48 s

0.8.0

desamis-no-MacBook-Air:cc senheng$ yarn add -D [email protected]
...success Saved 1 new dependency.
└─ [email protected]
...
[11:41:25] Starting 'js'...
[11:41:35] Finished 'js' after 10 s

@Blinnikov
Copy link

Has it been fixed in 0.9.0?

@schmod
Copy link
Owner

schmod commented Jun 4, 2018

Unlikely. 1.0.0 should be on its way as soon as I have some free time to test it, and may help with this somewhat. I know where the bottleneck is, but unfortunately need to be careful about any fixes, as the code in question was added to address a number of difficult-to-diagnose edge-cases where we weren't generating the right output.

@nasvillanueva
Copy link

@schmod Any news on this? I'd be happy to help if you can provide some information about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants