Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#3887 closed defect (worksforme)

Custom annotation for non-standard experiment settings

Reported by: Francesco Casella Owned by: Adeel Asghar
Priority: high Milestone: 1.11.0
Component: Testing Framework Version:
Keywords: Cc:

Description

If you select any option in the Simulation Setup of OMEdit which is not standardized in section 18.4 of the Modelica Specification 3.3r1 (i.e., anything besides StartTime, StopTime, Interval, Tolerance) and tick the checkbox "Save simulation settings inside model", those options are actually not saved. The next time the model is reloaded from file and simulated, those options will be re-set to default.

This is not a good idea: for instance, for non-stiff models using the default implicit solver dassl might introduce an unacceptable performance penalty, while an explicit solver could end up being much faster and less memory-consuming. It's really annoying if one has to set all those options manually every time.

I would suggest to introduce omc-specific vendor annotations to store all those extra settings.

Change History (12)

comment:1 by Francesco Casella, 9 years ago

See also #3488

comment:2 by massimo ceraolo, 9 years ago

see also #3859

comment:3 by Christoph <buchner@…>, 9 years ago

Would a custom annotation also be a good place to define a default set of plots?

E.g. when I have a complex model that I am simulating and adapting/enlarging, every time I open the model in OMEdit, after first simulation, I have to hunt down the quantities I want to plot in the variables browser. If you have O(10) variables distributed over multiple plots, this gets old after a while.

It would be awesome if,when working in OMEdit, I could put in the modelica source something like annotation(Plot(mymodel.component1.port_a.m_flow, mymodel.component2.pressure, mymodel.controlsignal), ParametricPlot(mymodel.component3.T, mymodel.overall_efficiency)). Is there already a way to achieve this, that I failed to identify? Kinda feels like this [cs]hould be part of the experiment annotation, but it isn't, afaict.

comment:4 by Francesco Casella, 9 years ago

Milestone: 2.0.01.10.0
Owner: changed from somebody to Per Östlund
Status: newassigned

Adrian told me he would talk to Per if he has some time to deal with this. Per, do you?

comment:5 by Adrian Pop, 9 years ago

Owner: changed from Per Östlund to Adeel Asghar

comment:6 by Adrian Pop, 9 years ago

We talked internally and Adeel and Arun will work on this.

comment:7 by Adrian Pop, 9 years ago

Component: *unknown*OMEdit

comment:8 by Adrian Pop, 9 years ago

Component: OMEditTesting Framework

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

Milestone: 1.10.01.11.0

Ticket retargeted after milestone closed

comment:10 by Francesco Casella, 8 years ago

Resolution: worksforme
Status: assignedclosed

It seems that this has been possible for a while, after #3488 was fixed, so I'll close this ticket.

comment:11 by Christoph Buchner <buchner@…>, 8 years ago

Should I put the plot annotation proposed in comment:3 into a separate issue?

comment:12 by Adeel Asghar, 8 years ago

Yes please create a new ticket for it.
Note that custom annotation should have a name like __OpenModelica_plot instead of Plot.

Note: See TracTickets for help on using tickets.