Opened 12 years ago
Last modified 6 years ago
#2153 new defect
Better analysis for function completion
Reported by: | Adrian Pop | Owned by: | Per Östlund |
---|---|---|---|
Priority: | high | Milestone: | 2.1.0 |
Component: | New Instantiation | Version: | trunk |
Keywords: | Cc: |
Description
We need way better analysis to find out which functions are complete and can be called during compilation.
Currently we consider non-complete:
- partial functions
- replaceable functions
but this is rather wrong and not really correct.
Unfortunately the Modelica Specification does not say anything about this, maybe we should add a ticket about it in the Modelica Trac.
Change History (12)
comment:3 by , 10 years ago
Milestone: | 1.9.1 → 1.9.2 |
---|
This ticket was not closed for 1.9.1, which has now been released. It was batch modified for milestone 1.9.2 (but maybe an empty milestone was more appropriate; feel free to change it).
comment:4 by , 10 years ago
Milestone: | 1.9.2 → 1.9.3 |
---|
Milestone changed to 1.9.3 since 1.9.2 was released.
comment:9 by , 8 years ago
Milestone: | 1.11.0 → 1.12.0 |
---|
Milestone moved to 1.12.0 due to 1.11.0 already being released.
follow-up: 11 comment:10 by , 7 years ago
Component: | Frontend → New Instantiation |
---|---|
Milestone: | 1.12.0 → 2.0.0 |
Owner: | changed from | to
@perost, is this still relevant?
comment:11 by , 7 years ago
Replying to casella:
@perost, is this still relevant?
Maybe? We haven't implemented function evaluation in the new instantiation yet, but I guess we do need to make sure any functions being used is valid. I don't really see what the issue would be though.
comment:12 by , 6 years ago
Milestone: | 2.0.0 → 2.1.0 |
---|
Actually, the only time we need to evaluate functions is for structural parameters/constants.
For all the other we can just leave call into place. The only issue might be with package constants, but we could have the package constants in the DAE, i.e.
and use this name everywhere where is needed.
Hopefully we'll do something like this in the new inst.