Opened 6 years ago

Closed 6 years ago

#4771 closed discussion (fixed)

Attribute dimension syntax

Reported by: Christian Kral <dr.christian.kral@…> Owned by: adeas31
Priority: high Milestone: 1.13.0
Component: OMEdit Version:
Keywords: Cc:

Description

Please consider the following model

model TestDimensions
  Modelica.Electrical.Analog.Basic.Ground ground1[3] annotation(
    Placement(visible = true, transformation(origin = {0, 0}, 
    extent = {{-10, -10}, {10, 10}}, rotation = 0)));
equation
  annotation(uses(Modelica(version = "trunk")));
end TestDimensions;

If I show the attributes of ground1 in the Diagram View, the field Dimensions shows: {3} (see attachment)

I wonder why braces are used in the dimensions field instead of square brackets [3], as square brackets are used in the Modelica code, too. I am not sure if this is a bug or a feature.

Attachments (1)

TestDimensions.png​ (41.2 KB) - added by Christian Kral <dr.christian.kral@…> 6 years ago.
Attribute window

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by Christian Kral <dr.christian.kral@…>

Attribute window

comment:1 Changed 6 years ago by Christian Kral <dr.christian.kral@…>

The field name in the PNG file should have been ground1, not ground -- sorry.

comment:2 follow-up: Changed 6 years ago by Christian Kral <dr.christian.kral@…>

Ticket title shall be dimension instead of dimenion

comment:3 Changed 6 years ago by Christian Kral <dr.christian.kral@…>

  • Summary changed from Attribute dimenion syntax to Attribute dimension syntax

comment:4 in reply to: ↑ 2 Changed 6 years ago by Christian Kral <dr.christian.kral@…>

Replying to Christian Kral <dr.christian.kral@…>:

Ticket title shall be dimension instead of dimenion

Just fixed the title (didn't know I have permission to change the title)

comment:5 Changed 6 years ago by adeas31

  • Milestone changed from Future to 1.13.0
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in 86eb441/OMEdit.

Note: See TracTickets for help on using tickets.