OMEdit - better omeditcommands.log

[Adrian Pop]

I would be really cool to be able to run omeditcommands.log by just renaming
it to .mos. Currently is not possible because you have a lot of text around
the commands you sent to omc and the answer from OMC.

getNthConnection(Modelica.Fluid.Examples.Tanks.ThreeTanks, 5) 15:58:36:932
{pipe2.port_b,tank2.ports[1], ""} 15:58:36:932
Elapsed Time :: 0 secs

getClassRestriction(Modelica.Fluid.Pipes.StaticPipe) 15:58:36:933
"model" 15:58:36:933
Elapsed Time :: 0 secs

existClass(Modelica.Fluid.Pipes.StaticPipe) 15:58:36:933
true 15:58:36:934
Elapsed Time :: 0.001 secs

getClassRestriction(Modelica.Fluid.Vessels.OpenTank) 15:58:36:934
"model" 15:58:36:934
Elapsed Time :: 0 secs

I suggest outputting the command you send to omc followed by "; // " then the time. Also the answer you get from OMC should be in comments:

/*
OMC answered: ....
*/

This way you can run an omeditcomands.log with OMC and debug way easier.
Alternatively you could generate an extra file with just the commands you
sent to omc followed by ";" named omeditcommands.mos

[Adrian Pop]

Currently I need to copy the commands by hand in another file or remove the extra text and that's just crazy time consuming :)

[Adeel Asghar]

A new file omeditcommands.mos will be created from r22655.

[Adeel Asghar]

Do you want me to add getErrorString(); after every command?

[Adrian Pop]

Yes please, but on the same line:

commandToOMC; getErrorString();

That would be perfect.

[Adeel Asghar]

Done in r22658.

Btw the omeditcommands.log file name is changed to omeditcommunication.log.

[Adrian Pop]

Thanks!

[Adrian Pop]

Just one minor issue, when you're creating a class you do:

existClass(FluidProblems); getErrorString();
model FluidProblems end FluidProblems;; getErrorString();

Note that you have two semicolons ;; one after the
other which makes the omeditcommands.mos script invalid.
Maybe you should check if there is already a ; don't
add another one.

[Adrian Pop]

Actually when you're sending strings to OMC in the script you need to use loadString
as model F end F; directly in the script does not work. So it should be:

existClass(FluidProblems); getErrorString();
loadString("model FluidProblems end FluidProblems;"); getErrorString();

in the omeditcommands.mos script to make it usable.

[Adeel Asghar]

Done in r23175.

[Dietmar Winkler]

It doesn't make sense to keep closed ticket in the "Future" milestone that were simply forgotten to assign to the correct milestone in the past.

[Martin Sjölund]

Removing the pre1.9.4 milestone in favor of 1.9.4.