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

Allow more eventvalues for events #3791

Closed
2 tasks
TD-er opened this issue Oct 3, 2021 · 4 comments · Fixed by #4004
Closed
2 tasks

Allow more eventvalues for events #3791

TD-er opened this issue Oct 3, 2021 · 4 comments · Fixed by #4004
Labels
Category: Rules Related to the rule engine Type: Enhancement Improve something already present

Comments

@TD-er
Copy link
Member

TD-er commented Oct 3, 2021

As requested on the forum

ToDo:

  • Allow for N %eventvalueN% (not limited?)
  • Replace unavailable event values with some default value (0 ???)
@TD-er TD-er added Category: Rules Related to the rule engine Type: Enhancement Improve something already present labels Oct 3, 2021
@Micha-he
Copy link
Contributor

Micha-he commented Oct 3, 2021

#3741

@TD-er
Copy link
Member Author

TD-er commented Oct 3, 2021

Thanks, that was indeed the issue I had in mind for the 2nd point.

@tonhuisman
Copy link
Contributor

Replace unavailable event values with some default value (0 ???)

Should probably be a setting (not sure what category it should go into though).
And maybe also add a command to set that default? Like: defaulteventvalue,N

@TD-er
Copy link
Member Author

TD-er commented Oct 3, 2021

We can also add a command like "letcheck" (or something with a better name) to not assume default values, but check for special values (also to check for NaN etc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Rules Related to the rule engine Type: Enhancement Improve something already present
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants