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

Combine RelationalOperator with NumericalMeasure #16

Open
ilkkarinne opened this issue Jun 7, 2018 · 0 comments
Open

Combine RelationalOperator with NumericalMeasure #16

ilkkarinne opened this issue Jun 7, 2018 · 0 comments
Assignees

Comments

@ilkkarinne
Copy link
Contributor

Relational operators (above or below the given value) are used in several places in the message classes as separate properties. Combining them as an Optional relational operator within the NumericalMeasure would be their use clearer since the entire value would them be accessible from a the single property, and the using code would not so easily forget to check the possible additional relational operator.

@ilkkarinne ilkkarinne self-assigned this Jun 7, 2018
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

1 participant