Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Named arguments for PulseAtoms #68

Open
Dominik-Vogel opened this issue Mar 15, 2018 · 0 comments
Open

Named arguments for PulseAtoms #68

Dominik-Vogel opened this issue Mar 15, 2018 · 0 comments

Comments

@Dominik-Vogel
Copy link
Member

Allow named arguments for PulseAtoms.
Current call signature

BluePrint.insertSegment(pos, func, args=(), dur=None, name=None, durs=None)

does not allow to use PulseAtoms with named arguments. I my opinion the named arguments should even be the default in order to avoid human error when calling a method like

bp.insertSegment(-1, monteCarloZeta, args=(12.3, 12.5, 59.76, 21.4 , 43.56, 3*pi))
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant