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:1 by Adrian Pop, 12 years ago

Actually, the only time we need to evaluate functions are 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.

omcglobal.packagename.path.to.constant.name

and use this name everywhere where is needed.
Hopefully we'll do something like this in the new inst.

Version 0, edited 12 years ago by Adrian Pop (next)

comment:2 by Martin Sjölund, 11 years ago

Milestone: 1.9.01.9.1

Postponed until 1.9.1

comment:3 by Martin Sjölund, 10 years ago

Milestone: 1.9.11.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 Martin Sjölund, 10 years ago

Milestone: 1.9.21.9.3

Milestone changed to 1.9.3 since 1.9.2 was released.

comment:5 by Martin Sjölund, 9 years ago

Milestone: 1.9.31.9.4

Moved to new milestone 1.9.4

comment:6 by Martin Sjölund, 9 years ago

Milestone: 1.9.41.9.5

Milestone pushed to 1.9.5

comment:7 by Martin Sjölund, 9 years ago

Milestone: 1.9.51.10.0

Milestone renamed

comment:8 by Martin Sjölund, 8 years ago

Milestone: 1.10.01.11.0

Ticket retargeted after milestone closed

comment:9 by Martin Sjölund, 8 years ago

Milestone: 1.11.01.12.0

Milestone moved to 1.12.0 due to 1.11.0 already being released.

comment:10 by Francesco Casella, 7 years ago

Component: FrontendNew Instantiation
Milestone: 1.12.02.0.0
Owner: changed from somebody to Per Östlund

@perost, is this still relevant?

in reply to:  10 comment:11 by Per Östlund, 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 Francesco Casella, 6 years ago

Milestone: 2.0.02.1.0
Note: See TracTickets for help on using tickets.