Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#5854 closed defect (fixed)

NFConvertDAE.convertStateSelectAttribute got invalid StateSelect expression

Reported by: dariomangoni@… Owned by: Per Östlund
Priority: high Milestone: Future
Component: New Instantiation Version: v1.16.0-dev
Keywords: Cc:

Description

I'm really sorry not to have a minimal example for this case, but the new frontend gives a lot of these errors.

I cannot find an easy way to replicate the issues (that are not there with old frontend)

[C:/dev/OM64bit/OMCompiler/Compiler/NFFrontEnd/NFConvertDAE.mo:453:9-454:91:writable] Error: Internal error NFConvertDAE.convertStateSelectAttribute got invalid StateSelect expression StateSelect.prefer

using OpenModelica v1.16.0-dev-189-gf9a0b4816

Attachments (1)

TestConvertStateSelectAttribute3.mo (3.5 KB ) - added by dariomangoni@… 5 years ago.
Almost minimal example

Download all attachments as: .zip

Change History (7)

comment:1 by Francesco Casella, 5 years ago

Can you share a non-minimal example, also privately? Otherwise it's very hard to find a fix.

comment:2 by Francesco Casella, 5 years ago

Component: *unknown*New Instantiation
Owner: changed from somebody to Per Östlund

by dariomangoni@…, 5 years ago

Almost minimal example

comment:3 by dariomangoni@…, 5 years ago

In the attachment you will find a Test model that should replicate the error.
It has been a really long work. I had to get rid of any sensible information/code and because of this the model will not run but nonetheless it throws the same exact error that also the complete model throws.

Please let me know if anything can be done to fix the issue.

in reply to:  3 comment:4 by Per Östlund, 5 years ago

Replying to dariomangoni@…:

In the attachment you will find a Test model that should replicate the error.

Thank you, the issue should be fixed in PR #736.

For frontend issues like this it's not necessary to have a "correct" model, it just needs to trigger the same error. I appreciate when users give me as minimal models as possible since it saves me the trouble of having to cut the model down myself (the test model I added in the PR is an even smaller version of your model in fact).

comment:5 by Per Östlund, 5 years ago

Resolution: fixed
Status: newclosed

Fixed in d3c77b8.

comment:6 by Francesco Casella, 5 years ago

Thanks to the reporter and to the fixer!

MWEs (in fact, MNWEs in this case) are essential for efficient debugging.

Note: See TracTickets for help on using tickets.