8000 Performance Analysis: · Issue #24 · power-assert-js/power-assert-formatter · GitHub
[go: up one dir, main page]
Skip to content

Performance Analysis: #24

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.

Sign up for GitHub

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

Jump to bottom
Open
jamestalmage opened this issue Nov 24, 2015 · 3 comments
Open

Performance Analysis: #24

jamestalmage opened this issue Nov 24, 2015 · 3 comments

Comments

@jamestalmage
Copy link
Contributor 


Start time: (2015-11-24 06:05:00 UTC) [treshold=1%,sorted]
# [order]  module                        time  %
   1 [50]  power-assert-...er/index.js)  38ms  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 86%
   2 [49]  ./lib/create...b/create.js)   37ms  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 84%
   3 [33]  ./traverse (n...traverse.js)  21ms  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 48%
   4 [23]  acorn (node_m...st/acorn.js)  15ms  ▇▇▇▇▇▇▇▇▇▇ 34%
   5 [17]  stringifier (...er/index.js)   6ms  ▇▇▇▇ 14%
   6  [7]  chalk (node_m...lk/index.js)   4ms  ▇▇▇ 9%
   7 [36]  ./udiff (node...ib/udiff.js)   4ms  ▇▇▇ 9%
   8 [16]  ./strategies...rategies.js)    3ms  ▇▇ 7%
   9 [22]  estraverse (n...traverse.js)   3ms  ▇▇ 7%
  10 [35]  googlediff (n...ff/index.js)   3ms  ▇▇ 7%
  11 [34]  ./javascript/...mpressed.js)   2ms  ▇▇ 5%
  12 [20]  ./string-writ...g-writer.js)   1ms  ▇ 2%
  13 [21]  ./package.jso...ackage.json)   1ms  ▇ 2%
  14 [10]  xtend (node_m...mmutable.js)   1ms  ▇ 2%
  15  [4]  ansi-styles (...i-styles.js)   1ms  ▇ 2%
  16 [14]  array-reduce-...ht/index.js)   1ms  ▇ 2%
  17 [30]  ./esnode (nod...b/esnode.js)   1ms  ▇ 2%
  18 [31]  array-foreach...b/esnode.js)   1ms  ▇ 2%
  19  [5]  strip-ansi (n...si/index.js)   1ms  ▇ 2%
  20  [8]  traverse (nod...se/index.js)   1ms  ▇ 2%
  21 [18]  eastasianwidt...ianwidth.js)   1ms  ▇ 2%
  22 [19]  ./string-widt...ng-width.js)   1ms  ▇ 2%
  23 [39]  xtend (node_m...-options.js)   1ms  ▇ 2%
  24 [41]  ./built-in/as...ssertion.js)   1ms  ▇ 2%
  25 [44]  ./built-in/di.../diagram.js)   1ms  ▇ 2%
  26 [47]  array-foreach...ray-foreach)   1ms  ▇ 2%
  27 [48]  ./built-in/bi...pression.js)   1ms  ▇ 2%
  28 [24]  acorn-es7-plu...7-plugin.js)   1ms  ▇ 2%
Total require(): 51
Total time: 44ms

screenshot 2015-11-24 01 02 35

Biggest Offenders Are:

  • Acorn 15ms
  • Stringifier 6ms
  • Chalk 4ms (which I am guessing is a stringifier dependency).

Since AVA will print in the main thread, not the forked thread, is there a way to disable ever require-ing chalk?
@jamestalmage
Copy link
Contributor Author

Better yet - lets figure out how to get power-assert-formatter out of each forked AVA process entirely.

We can spit the relevant information up to the master thread, and have it print it out.

It would still be good to see acorn go.

@twada
Copy link
Member
twada commented Nov 24, 2015

Chalk 4ms (which I am guessing is a stringifier dependency).

I don't use chalk. It's a dependency of time-require itself.

@jamestalmage
Copy link
Contributor Author

cool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@twada @jamestalmage
0