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

Cannot enlarge memory arrays. #51

Open
Christian-Yang opened this issue May 10, 2017 · 13 comments
Open

Cannot enlarge memory arrays. #51

Christian-Yang opened this issue May 10, 2017 · 13 comments

Comments

@Christian-Yang
Copy link

Christian-Yang commented May 10, 2017

after npm install , I'm running /root/node-v6.9.0-linux-x64/bin/ngd , then get errors as follow:

1

2

then i use /root/node-v6.9.0-linux-x64/bin/ngd -s TOTAL_MEMORY=60000000 ,still get errors.

how to solve this problem?

THANKS VERY MUCH!

@BenoitFroment
Copy link

same here

@manekinekko
Copy link
Member

this issue comes from the Viz dependency. We're trying another approach. Will keep you updated.

@manekinekko
Copy link
Member

we're experimenting an other approach which would be rendering the graph on the client side (taking into account some performances issues).

@wgrabowski
Copy link

@manekinekko is this experimental approach somehow available (branch or something)?

@pantonis
Copy link

Have the same issue here? This thread is 10 months old. Any news?

@ghost
Copy link

ghost commented Apr 13, 2018

Same issue. Any news?

@manekinekko
Copy link
Member

manekinekko commented May 5, 2018

@vogloblinsky is working on a possible solution to fix this.
To give you more context, this "solution" will allow rendering the graph deps on the client side (the user's browser). In fact, the library we depend on viz.js doesn't deal well with large projects. It simply crashes. In addition, it's not maintained anymore.

In the meantime, we've included a "workaround" that simply skip the graph that are very large and don't render. You still should have the rest of the documentation generated and working properly (but without the affected graphs).

To do so, just run compodoc via a node script:

package.json
{
   "scripts": {
      "reports:doc": "node ./tools/scripts/compodoc.js"
   }
}

tools/scripts/compodoc.js

const { spawn } = require('child_process');
const cmd = spawn('compodoc', ['-p', 'src/tsconfig.app.json', '-d', 'reports/documentation/']);
cmd.stdout.pipe(process.stdout);
cmd.stderr.pipe(process.stdout);

Doing so will trick your CI and not mark your build as FAILED.

@DaSchTour
Copy link

DaSchTour commented Jun 21, 2018

@manekinekko I tried your solution but it doesn't work. It just throws an error

events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: spawn compodoc ENOENT
    at _errnoException (util.js:1022:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
    at onErrorNT (internal/child_process.js:372:16)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)
    at Function.Module.runMain (module.js:695:11)
    at startup (bootstrap_node.js:188:16)
    at bootstrap_node.js:609:3

I also have no idea how this should work because compodoc is not registered as a binary anywere.

@vogloblinsky
Copy link
Contributor

@DaSchTour the trick from @manekinekko is for compodoc installed globally in your system.

npm i -g @compodoc/compodoc

@AhHa45
Copy link

AhHa45 commented Dec 4, 2018

so this tool is useless?

@AhHa45
Copy link

AhHa45 commented Dec 5, 2018

running compodoc via a node script worked for me.

@cmxl
Copy link

cmxl commented Feb 12, 2020

any update on this?

@haris-begluk
Copy link

Same thing on windows, can't generate main diagram.

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

No branches or pull requests

10 participants