Invalid error message from OM

[massimo ceraolo]

Steps to reproduce:

• load omEdit
• load Test.mo
• Change Amplitude of sine wave in E1 to 101
• Change Amplitude of sine wave in E1 to 100

The following error message appears:

[3] 10:07:51 Syntax Error
[: 1:0-1:0]: No viable alternative near token: <EOF>

However, the code is ok and simulates well.
(report by my student Vannuccini)

[Adrian Pop]

I can't reproduce this with a bit older OMEdit, I will compile a new one.

[massimo ceraolo]

We did this with 1.17.0-dev-141

[Francesco Casella]

@ceraolo, I tried with the latest nightly build v1.17.0-dev-195-g637d5adedc, I can't reproduce this either. Would you mind trying that out?

[massimo ceraolo]

I will tomorrow

[Francesco Casella]

@ceraolo, if some of your students spot issues, you can put their email in the cc: field of the ticket, so they can get and provide feedback faster.

[massimo ceraolo]

Replying to casella:

@ceraolo, if some of your students spot issues, you can put their email in the cc: field of the ticket, so they can get and provide feedback faster.

I grant them marks if they found replicable issues.
They are at their first experiences with OM, and are not ready for trac: I tried what you mention two years ago with poor results.

[massimo ceraolo]

Replying to casella:

@ceraolo, I tried with the latest nightly build v1.17.0-dev-195-g637d5adedc, I can't reproduce this either. Would you mind trying that out?

I downloaded and run dev-195 and got exactly the same issue.
I made all tests under Windows 64 bit.

Consider that the same issue was observed on three different PC's (the student's and two of mine)

[Adeel Asghar]

I also can't reproduce this.

@ceraolo did you make the change via the GUI or in the text view? I tried all combinations. Changing amplitude of all SineVoltages from parameters window and from text view and none produced that error.

[Francesco Casella]

Replying to adeas31:

I also can't reproduce this.

Me neither.

Do you have some particular setting that you recommend to your students?

@adeas31, is there any file we can get from @ceraolo which contains all the OMEdit settings?

[massimo ceraolo]

I enclose the settings we use (OMedit settings.pdf), with evidence given to some differences from the default. I suppose there are no differences from default outside the shown pictures. I enclose the log as well.

[massimo ceraolo]

Replying to adeas31:

I also can't reproduce this.

@ceraolo did you make the change via the GUI or in the text view? I tried all combinations. Changing amplitude of all SineVoltages from parameters window and from text view and none produced that error.

Did the changes via GUI (parameters window)

[Adeel Asghar]

Ah now I see why this is happening.

Enable replaceable support and you won't see the error. I will fix this.

[Adeel Asghar]

Fixed in ebd255f/OpenModelica and ported to 1.16

[Francesco Casella]

What has replaceable support to do with changing a parameter value?

[Francesco Casella]

I'll keep this open until we release 1.16.2 or 1.17.0.

[Adrian Pop]

Replying to casella:

What has replaceable support to do with changing a parameter value?

We based *some* of the new changes on the "replaceable" prefix of the component or class which was true even for some replaceable components. So some of the new parts were ran even if the replaceable support flag was not activated which generated this error.

Now all the new parts are guarded with the replaceable support flag.

[Francesco Casella]

ok, thanks!

[Francesco Casella]

Replying to casella:

I'll keep this open until we release 1.16.2 or 1.17.0.

Sorry, this was stupid, we always close the ticket when the fix is comitted to the corresponding version branch.