Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect x-axis label when plotting arrays #8784

Closed
casella opened this issue Mar 29, 2022 · 1 comment · Fixed by #8826 or #8828
Closed

Incorrect x-axis label when plotting arrays #8784

casella opened this issue Mar 29, 2022 · 1 comment · Fixed by #8826 or #8828
Assignees
Labels
COMP/GUI/OMEdit Issue and pull request related to OMEdit
Milestone

Comments

@casella
Copy link
Contributor

casella commented Mar 29, 2022

Steps to Reproduce

Run the following model

model Test
  Real x[10] = sin(linspace(0, 6.28, 10));
end Test;

in OMEdit, then plot the whole x array.
immagine

The label on the x-axis is index (s), which is quite confusing, since the unit (s) is completely wrong.

Expected Result

I would suggest to write array element index instead.

@casella casella added the COMP/GUI/OMEdit Issue and pull request related to OMEdit label Mar 29, 2022
@casella casella added this to the 1.19.0 milestone Mar 29, 2022
@casella casella added this to To do in 1.19.0 Release plan via automation Mar 29, 2022
adeas31 added a commit to adeas31/OpenModelica that referenced this issue Apr 12, 2022
adeas31 added a commit to adeas31/OpenModelica that referenced this issue Apr 12, 2022
1.19.0 Release plan automation moved this from To do to Done Apr 12, 2022
adeas31 added a commit that referenced this issue Apr 12, 2022
@casella
Copy link
Contributor Author

casella commented Apr 12, 2022

@adeas31, in fact there is another small issue here. As we know, arrays indeces in Modelica start from one, not from zero. The plot should actually reflect this. For example, in the MWE shown above, the indeces should go from 1 to 10, not from 0 to 9.

@casella casella reopened this Apr 12, 2022
1.19.0 Release plan automation moved this from Done to In progress Apr 12, 2022
adeas31 added a commit to adeas31/OpenModelica that referenced this issue Apr 12, 2022
1.19.0 Release plan automation moved this from In progress to Done Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMP/GUI/OMEdit Issue and pull request related to OMEdit
Projects
No open projects
2 participants