napajs icon indicating copy to clipboard operation
napajs copied to clipboard

Fail to run examples by (node >=v8.5.0), so please try napa examples by (node <= v8.4.0) before fix

Open jmcpeak opened this issue 7 years ago • 8 comments

A few times it did work without error....

I got these errors in both the pi and fibonacci examples....

System: Mac 10.11.6 NodeJS v8.7.0 NPM v5.5.1

screen shot 2017-10-23 at 9 58 11 am

screen shot 2017-10-23 at 10 01 56 am

jmcpeak avatar Oct 23 '17 13:10 jmcpeak

I have the same issue any suggestion is welcome

Thanks

syahiaoui avatar Oct 30 '17 10:10 syahiaoui

I got the same issue. Sometimes it works, most of the time: segment fault.

[6112:0x103061600]      252 ms: [Finished reentrant Scavenger during Scavenge.]
[1]    6112 segmentation fault  node estimate-pi-in-parallel.js

tuananh avatar Oct 30 '17 13:10 tuananh

thx for the feedback, and we are working on it as P1 item.

helloshuangzi avatar Oct 30 '17 15:10 helloshuangzi

Whene i use LTS version(v6.11.5) of node it works fine: v6.9.0 v6.11.3 -> v6.11.5 v8.5.0 v8.6.0 v8.8.1 system default -> v8.6.0 node -> stable (-> v8.8.1) (default) stable -> 8.8 (-> v8.8.1) (default) iojs -> N/A (default) lts/* -> lts/boron (-> v6.11.5) lts/argon -> v4.8.5 (-> N/A) lts/boron -> v6.11.5

the results: Nth Fibonacci # of workers latency in MS ----------------------------------------------------------- 10 55 4 20 11 89 4 22 12 144 4 29 13 233 4 45 14 377 4 53 15 610 4 82 16 987 4 143

syahiaoui avatar Oct 31 '17 08:10 syahiaoui

@syahiaoui Thanks for the valuable feedback. We just verified it works well by node <= v8.4.0 on linux, and it will fail with random callstacks by node >=v8.5.0 We are working to resolve this issue as our top priority, and we contacted node.js team to accelerate it. So, please try napa examples with node <= v8.4.0 before that. Thank you very much.

helloshuangzi avatar Nov 01 '17 02:11 helloshuangzi

This is tracked in nodejs/node#16658.

daiyip avatar Nov 03 '17 19:11 daiyip

i have just upgraded to v9.6. i hope that library will support that soon

jjhesk avatar Mar 05 '18 17:03 jjhesk

@jjhesk there was a bug to invalidate the workaround for this issue on Node 8.10.0 with Napa.js 0.2.0. Please update to 0.2.1 and have a try.

fs-eire avatar Mar 12 '18 21:03 fs-eire