Synchronized execution of python script

[dhaval-lila]

Is there way trun python script in sync with node

function() {
 let pyshell = new PythonShell('.myscript.py')
let content = ""
 pyshell.on('message',function(message){
 content = message
 })
return content // content needs to be message instead due to async nature it returns empty string
}

[Almenon]

Look at the readme - you want the run command.

[dhaval-lila]

I am guessing you meant PythonShell.run() that one too is async function. It takes callback function.

[Almenon]

Oh yeah you're right, sorry. We don't have a synchronous option but that's a good idea

[dhaval-lila]

If you are looking to add this feature , I will gladly help and I was looking to contribute and this is great opportunity for my first contribution I am familiar with node and related web technologies so can you guide me through what i need to know to implement synchronous option

[Almenon]

Thanks! For sychronus operation first we need to migrate to using promises (read #77). I've created a v2 branch for this: https://github.com/extrabacon/python-shell/tree/version2

Once we are using promises then we can make sync versions of our functions - see this article: https://medium.com/@patarkf/synchronize-your-asynchronous-code-using-javascripts-async-await-5f3fa5b1366d

[WingDust]

I want to ask don't it already come true

[abrichr]

+1

[Almenon]

re-posting my comment from #77:

To be honest I'm probably not going to get to this anytime soon. I'd be happy to review a PR if someone submits it. Or if someone wants to donate to charity I can use that as motivation to get around to implementing this.

[nkhil]

We had the same issue today, and ended up promisifying it like so:

function someFunction(date, data){
 return new Promise((resolve, reject) => {
 let result;
 let pyshell = new PythonShell('pyshell_test.py', {mode: 'text', args: [date]});
 
 pyshell.send(JSON.stringify(data['someProperty']));
 
 pyshell.on('message', function (message) {
 result = JSON.parse(message);
 });
 
 pyshell.on('stderr', function (stderr) {
 console.log(stderr);
 });
 
 pyshell.end(function (err, code, signal) {
 if (err) reject(err);
 console.log('The exit code was: ' + code);
 console.log('The exit signal was: ' + signal);
 console.log('finished');
 resolve(result);
 });
 
 });
}

[karanbalani]

@nkhil this works (at least for now), thanks. :)

[AlexString]

@nkhil implementation worked for me but I did something more like this in a Class:

callPythonScript = () => {
	return new Promise((resolve, reject) => {
		PythonShell.run(this.pythonScriptName, this.options, (err, result) => {
			if (err) reject(err);

			resolve(result);
		});
	});
};

[dschweinbenz]

@nkhil implementation worked for me but I did something more like this in a Class:

callPythonScript = () => {
	return new Promise((resolve, reject) => {
		PythonShell.run(this.pythonScriptName, this.options, (err, result) => {
			if (err) reject(err);

			resolve(result);
		});
	});
};

One should use return reject(err) to avoid resolving the request afterwards. See https://stackoverflow.com/questions/32536049/do-i-need-to-return-after-early-resolve-reject

[dusan-maintains]

I'm actively maintaining this package and have an open PR (#320) fixing the runString() temp-path bug. I'm also tracking maintenance work publicly at dusan-maintains/oss-maintenance-log — all PRs and SLA are logged automatically every 6 hours.

If you're hitting issues with synchronized execution, the underlying problem is often related to how temp files are handled. Happy to look into this further if you can share a minimal repro.