Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

Improve memory usage #36

Open
jumpinjackie opened this issue Feb 18, 2016 · 2 comments
Open

Improve memory usage #36

jumpinjackie opened this issue Feb 18, 2016 · 2 comments

Comments

@jumpinjackie
Copy link
Owner

We seem to be allocating objects with reckless abandon to the point where node errors with ELIFECYCLE (ie. out of memory) when we try to generate a d.ts for Cesium.js

@jumpinjackie
Copy link
Owner Author

Initial plan of attack:

  • Add memory debugging facilities (node-memwatch)
  • Profile run against ol3 source
  • Remove the use of various classes in tsoutput.ts. These classes are nothing more than intermediate objects about the original JSDoc doclet. While convenient, I strongly believe the bulk of the memory usage is in storing these objects all over the place. Replace with utility methods that do the same thing off of the original doclets.
  • Re-profile run against ol3 source

@jumpinjackie
Copy link
Owner Author

Downgrading severity. ELIFECYCLE is not due to OOM, but because JSDoc actually reported errors.

That's not to say we can't improve the memory usage, but it doesn't appear to be the cause at the moment

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

No branches or pull requests

1 participant