Return values of make_xxx_pulse functions

[schuenke]

Since 7542b39 the make-pulse-functions like make_sinc_pulse.py and make_gauss_pulse.py return a different number of variables as before when using return_gz = False.

For compatibility with previous versions I suggest to return Nones for gz and gzr as it was the case before the release of v1.3.1. However, I think using the _return_gz' flag instead of the try block is a good idea and I suggest to change the return lines only. For example the make_sinc_pulse.py could be modified to:

if return_gz:
    return rf, gz, gzr
else:
    return rf, None, None

This would also solve issue #46

Let me know if I should include this in the PRs for issue #46 and issue #48.

[sravan953]

I guess the return function executes irrespective of the return_gz value, missed this! I will work on this fix, and incorporate your suggested changes, consequently fixing #46.

[sravan953]

Please let me know if pulling from the latest dev commit fixes this issue for you.

[sravan953]

Thanks @schuenke for bearing with me. The latest commits must have fixed this for good!

[schuenke]

Issue is fixed now. Thanks @sravan953 !

[schuenke]

Hey @sravan953,
will you merge this fix and the others from the dev branch into master and create a new release soon?

[sravan953]

Hi, yes! I commit to publishing a new release by Friday (04/30). Thanks for bearing with me!

[schuenke]

Will this be 1.3.1post1 than? We need to submit the proofs of our paper by tomorrow and I would lile to mention the release.

[sravan953]

Yes 1.3.1post1

[sravan953]

@schuenke Hi, published!

[schuenke]

Thanks @sravan953 . Will test it tomorrow 👍

[schuenke]

Seems to work fine. Thanks.