Skip to content

feat #335 configurable number of phantomjs instances in attester #337

Closed
wants to merge 1 commit into from

2 participants

@jakub-g
Aria Templates member
jakub-g commented Jan 18, 2013

Type in the console e.g.
npm config set ariatemplates:phantomjsInstances 4
to change the default to 4.

See #335.

This depends on the pull request in Attester. Before integrating, the version of Attester in package.json should be bumped and included in this pull request's commit.

@jakub-g
Aria Templates member
jakub-g commented Jan 18, 2013

Depends on attester/attester#8

@jakub-g jakub-g feat #335 configurable number of PhantomJS instances in Attester
Makes it possible to increase (locally, per-user) no. of PhantomJS
instances in Attester to parallelize the execution of the test suite
when launched via 'npm run-script attester' (and 'npm test').

Type in the console e.g.
 npm config set ariatemplates:phantomjsInstances 4
to change the default to 4.

Set it to ~= your processor's noOfCores * noOfThreads * 2
for max performance. E.g. on quad-core dual-threaded processor,
set it to 4*2*2 = 16.

Keep in mind that each thread consumes ~= 30-50 MB of RAM.

This commit requires attester 1.0.2.

Close #335. Close #337.
be8dd4b
@divdavem divdavem was assigned Jan 24, 2013
@jakub-g jakub-g added a commit that closed this pull request Jan 24, 2013
@jakub-g jakub-g feat #335 configurable number of PhantomJS instances in Attester
Makes it possible to increase (locally, per-user) no. of PhantomJS
instances in Attester to parallelize the execution of the test suite
when launched via 'npm run-script attester' (and 'npm test').

Type in the console e.g.
 npm config set ariatemplates:phantomjsInstances 4
to change the default to 4.

Set it to ~= your processor's noOfCores * noOfThreads * 2
for max performance. E.g. on quad-core dual-threaded processor,
set it to 4*2*2 = 16.

Keep in mind that each thread consumes ~= 30-50 MB of RAM.

This commit requires attester 1.0.2.

Close #335. Close #337.
9446154
@jakub-g jakub-g closed this in 9446154 Jan 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.