multi-tasks 1.2.3 → 1.2.4

package/README.md

@@ -43,7 +43,7 @@ multiTasks({
     processTask, 
     multi_task_parent_folder: `C:/tmp-demo`, //a directory to store progress and results files, you can check the progress here
     quotaOfEachTask:1, //The number of subtasks assigned to each worker at one time, the default is 1
-    numberOfWorks: 6 //Assign how many workers are working in parallel
+    numberOfWorks: 6 //Assign how many workers are working in parallel, default is the number of cpu cores minus one
 });
 
 ```
@@ -71,6 +71,7 @@ node examples/example1/run
 
 Changelog:
 
+- 1.2.4 Fix: opt.numberOfWorks not work
 - 1.2.3 Update README
 - 1.2.2 Update README and examples
 - 1.2.1 Handle exceptions and errors in subtasks

package/package.json

@@ -1,21 +1,22 @@
 {
   "name": "multi-tasks",
-  "version": "1.2.3",
+  "version": "1.2.4",
   "description": "",
   "main": "index.js",
   "directories": {
     "example": "example"
   },
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "jest"
   },
   "keywords": [
     "multitask",
     "worker",
     "multithread"
   ],
-  "dependencies": {
-  },
   "author": "zhanglei923@gmail.com",
-  "license": "MIT"
+  "license": "MIT",
+  "devDependencies": {
+    "jest": "^29.7.0"
+  }
 }

package/test/splitter.test.js

@@ -0,0 +1,17 @@
+let splitter = require('../workers/taskSplitter');
+
+let createArray = (len)=>{  
+  let alltasks = []
+  for(let i=0;i<len;i++){
+      alltasks.push(i)
+  }
+  return alltasks;
+}
+
+test('test taskSplitter', async () => {
+    
+  expect(splitter.split(createArray(3), 5).length).toBe(1);
+  expect(splitter.split(createArray(16), 5).length).toBe(4);
+  expect(splitter.split(createArray(101), 1).length).toBe(101);
+  expect(splitter.split(createArray(101), 33).length).toBe(4);
+});

package/workers/master.js

@@ -37,11 +37,11 @@ let hasWorker = ()=>{
   return workidlist.size !== 0;
 }
 let run = (currentProducer, currentConsumer, opt)=>{
-    if(typeof opt === 'undefined') opt = {
-      quotaOfEachTask: 10
-    };    
+    let defaultCPUs = numCPUs - 1;
+    if(typeof opt === 'undefined') opt = {};    
     let masterId = currentProducer.getTaskName();
-    if(typeof opt.numberOfWorks === 'undefined') opt.numberOfWorks = numCPUs;
+    if(typeof opt.quotaOfEachTask === 'undefined') opt.quotaOfEachTask = 1;
+    if(typeof opt.numberOfWorks === 'undefined') opt.numberOfWorks = defaultCPUs;
     console.log('master1, masterId=', masterId)
 
 
@@ -155,8 +155,7 @@ let start = (ProducerClass, ConsumerClass, taskConfig)=>{
   producer.setTaskTrunkInfo(taskConfig);
   consumer.setTaskTrunkInfo(taskConfig);
   run(producer, consumer, {
-      quotaOfEachTask:2,
-      numberOfWorks: 6
+      ...taskConfig
   });
 }
 module.exports = {