Opened 6 years ago

Closed 6 years ago

#5137 closed defect (worksforme)

Wrong choice of angle unit in phase of sine wave

Reported by: bebi@… Owned by: adeas31
Priority: high Milestone: 1.13.0
Component: OMEdit Version: v1.13.0-dev-nightly
Keywords: Sine unit degrees Cc: mads.nannestad@…

Description

Hi,
When inserting the Modelica.Blocks.Sources.Sine and modifying the parameters from the "parameters" window in OpenModelica, it seems that one can express the phase either in degrees or in radians.
In reality, the choice of the angle unit is not considered, and the angle should always be provided in radians.
We suggest to either add the option of converting the angle, or to remove the choice of the angle unit since it is misleading.
Attached a small example where the block was inserted and the phase was specified as 90 "deg". One can see, that the information about the degrees is lost.
Thank you, best regards.
Mads and Benoit

Attachments (1)

TestSine.mo (480 bytes) - added by bebi@… 6 years ago.
Test file to open with OMEdit

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by bebi@…

Test file to open with OMEdit

comment:1 Changed 6 years ago by casella

  • Milestone changed from Future to 1.13.0
  • Resolution set to worksforme
  • Status changed from new to closed
  • Version changed from v1.12.0 to v1.13.0-dev-nightly

I tried with the latest nightly build and the problem seems to be fixed. I would suggest you to install the 1.13.0 stable development version, which should be free of this issue. If you still have problems, feel free to reopen the ticket.

Note: See TracTickets for help on using tickets.