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

TimeTable does not assert that first time value is 0 #2209

Closed
henrikt-ma opened this issue Mar 9, 2017 · 1 comment
Closed

TimeTable does not assert that first time value is 0 #2209

henrikt-ma opened this issue Mar 9, 2017 · 1 comment
Assignees
Labels
L: Blocks Issue addresses Modelica.Blocks
Milestone

Comments

@henrikt-ma
Copy link
Contributor

The TimeTable documentation clearly states that the first time value must be 0. However, there is no corresponding assert in the implementation, and failing to satisfy the requirement leads to array indexing out of bounds, which at best will result in a generic out-of-bounds error message that won't be able to pinpoint the problem. In the worst case, the out-of-bounds isn't detected, and the table produces invalid values without any indication of a problem.

@beutlich beutlich added the L: Blocks Issue addresses Modelica.Blocks label Mar 9, 2017
@beutlich beutlich added this to the MSL_next-MINOR-version milestone Mar 9, 2017
@beutlich beutlich self-assigned this Mar 13, 2017
@beutlich
Copy link
Member

Resolved by 20fe8c9 in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: Blocks Issue addresses Modelica.Blocks
Projects
None yet
Development

No branches or pull requests

2 participants