Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Only clocked/sampled RandomRealSource #91

Open
tbeu opened this issue Aug 13, 2015 · 3 comments
Open

Only clocked/sampled RandomRealSource #91

tbeu opened this issue Aug 13, 2015 · 3 comments

Comments

@tbeu
Copy link
Collaborator

tbeu commented Aug 13, 2015

There actually is no need to have sampled RNG in Blocks.OperatingSystem.RandomRealSource. I propose to add a conditional external trigger port for the block. In case this trigger is off the RNG should happen in every step (which matches also the block icon). Do you agree?

@tbeu
Copy link
Collaborator Author

tbeu commented Aug 13, 2015

Same argument would hold for input devices.

Of course it would break the backward compatibility.

@bernhard-thiele
Copy link
Collaborator

@tbellmann Would that create any problems with existing code for you?

@tbellmann
Copy link

In most cases, the sample rate is left unchanged, so it wouldn't really that big of an issue. But I don't like external trigger ports in general, I would prefer a solution with a checkbox, like an "disableTrigger" parameter and no additional port. In this case, it also would be completely backward compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants