Changes between Initial Version and Version 1 of Ticket #6129, comment 7


Ignore:
Timestamp:
2020-09-25T17:56:55Z (5 years ago)
Author:
Francesco Casella

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #6129, comment 7

    initial v1  
    1 I tested the results of the PR today. Now all the relevant packages show up.
     1I tested the results of the PR today with the attached test case. Now all the relevant packages show up. However, there are still some more that should not belong there and should be filtered out
    22
    3 There are still some more that should not belong there and should be filtered out:
     3Partial classes, e.g. {{{Modelica.Media.Interfaces.PartialPureSubstance}}}), that would produce an error when the redeclare statement is later processed by the front end, should be removed.
    44
    5 - Partial packages, e.g. {{{Modelica.Media.Interfaces.PartialPureSubstance}}}), that would produce an error when the redeclare statement is later processed by the front end
    6 - Packages defined within models, e.g. all the {{{Modelica.Fluid}}} packages that show up in the drop-down menu, because they cannot be reached by lookup, according to [https://specification.modelica.org/v3.4/Ch5.html#composite-name-lookup Section 5.3.2] of the Modelica Specification.
     5In general, all names that include at least one partial class should be filtered out, since lookup in partial classes is not allowed in a simulation model.