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

Fix group operators #192

Closed
0joshuaolson1 opened this issue May 26, 2017 · 1 comment
Closed

Fix group operators #192

0joshuaolson1 opened this issue May 26, 2017 · 1 comment
Assignees

Comments

@0joshuaolson1
Copy link
Contributor

0joshuaolson1 commented May 26, 2017

https://picolabs.atlassian.net/wiki/display/docs/Group+Operators compiles and doesn't break the engine AFAICT, but the one test I did with repeat n doesn't use the current n when the ruleset using this feature is modified to have a larger n.

@farskipper
Copy link
Member

Once the rule matches it won't reset if you change n. However, if you set n=2 signaled one event, then changed it to 10 you'll need to signal 9 more events before it will evaluate the rule body.

This is b/c all select when state machines only remember their current state. Rather than using an event log to recover it's current state. If we provide a built in event log (see #17) we could fix this.

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

No branches or pull requests

2 participants