-
Notifications
You must be signed in to change notification settings - Fork 168
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
Make type explicit for fill operator #4393
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I see the point of this PR, since all these functions have a third input which is a subtype of Real; as Modelica defines automatic conversions from Integer to Real, is there any difference?
Besides, the default is actually an empty array.
That said, I don't see anything wrong with this change, even if I don't get the point 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@arunkumar-narasimhan can you please review this PR so that it gets merged in? @Esther-Devakirubai this can also be ported to maint/v4.1.x. Thanks! |
Backporting this to maint 4.1.x by #4462 |
This is derived from #4389. It helps tools to correctly deduce the array type, just by checking the argument of fill.
Note, that it already was a mixed usage of Integer zero and Real zero in Media functions.