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

Update icons in Modelica.Mechanics.MultiBody.Visualizers from Wolfram MathCore #3087

Merged
merged 5 commits into from
Sep 3, 2019

Conversation

beutlich
Copy link
Member

@beutlich beutlich commented Aug 9, 2019

Use nicer icons from Wolfram SystemModeler for

  • Modelica.Mechanics.MultiBody.Visualizers
  • Modelica.Mechanics.MultiBody.Visualizers.Ground
  • Modelica.Mechanics.MultiBody.Visualizers.Torus
  • Modelica.Mechanics.MultiBody.Visualizers.VoluminousWheel
  • Modelica.Mechanics.MultiBody.Visualizers.PipeWithScalarField
  • Modelica.Mechanics.MultiBody.Visualizers.Advanced.PipeWithScalarField

@maltelenz @henrikt-ma For MSL v3.2 the icons were designed by @d-hedberg at Wolfram MathCore. Is it okay for you to take these six new icons for regular MSL again?

Taken from Wolfram SystemModeler
@beutlich beutlich added the L: Mechanics.MultiBody Issue addresses Modelica.Mechanics.MultiBody label Aug 9, 2019
@beutlich beutlich added this to the MSL4.0.0 milestone Aug 9, 2019
@beutlich beutlich self-assigned this Aug 9, 2019
Copy link
Contributor

@maltelenz maltelenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, these are fine to use.

Copy link
Contributor

@henrikt-ma henrikt-ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine, by nature of being approved by @maltelenz.

@beutlich beutlich changed the title Update icons in Modelica.Mechanics.MultiBody.Visualizers Update icons in Modelica.Mechanics.MultiBody.Visualizers from Wolfram MathCore Aug 12, 2019
@tobolar
Copy link
Contributor

tobolar commented Aug 29, 2019

What is the actual reason for this changes? To prefer image definition inside of a class instead of by an external image?

@beutlich
Copy link
Member Author

No, to have nicer icons.

@tobolar
Copy link
Contributor

tobolar commented Aug 29, 2019

Considering rendering in Dymola's package browser, Joints icon seems not to be better/nicer to me. Other components are fine.

beutlich_better-icons_browser2

Dymola's diagram:

beutlich_better-icons

@maltelenz
Copy link
Contributor

maltelenz commented Aug 29, 2019

The rendering of the Joints icon looks fine in SystemModeler, and reasonable in OpenModelica, so I think this is a Dymola problem.
SystemModeler:
Screenshot_20190829_172152
OpenModelica:
image
Edit: Had the wrong version loaded for OpenModelica screenshot. Updated now, my comment still stands.
Edit 2: And here is the corresponding diagram with the new icons rendered by SystemModeler:
Screenshot_20190829_180308

@beutlich
Copy link
Member Author

Here's how the Joints icon looks in SimulationX.

MSL 3.2.3:
grafik

This PR:
grafik

I also prefer that we should stick with the old icon since drawing cylindric gradients seems problematic in small icons.

@beutlich
Copy link
Member Author

And here's how the other new icons look in SimulationX:

grafik

@tobolar
Copy link
Contributor

tobolar commented Aug 30, 2019

OK, let's focus on the Joints icon only. I'm going to prepare yet another atempt this morning and will let you know.

@tobolar
Copy link
Contributor

tobolar commented Aug 30, 2019

Here are my attempts to improve the icons. In most cases, Dymola renders the cylinders poorly. In Joints6 I've used the same principle like in Torus and VoluminousWheel, i.e. overlayed several polygons. This seems fine to me.

beutlich_better-icons_Joints_browser2

@maltelenz, @beutlich: Please check in other tools whether this could be a solution.
https://gist.github.com/tobolar/95e62615d30ed5931cfd94c245040756

@maltelenz
Copy link
Contributor

maltelenz commented Aug 30, 2019

I added a 7 that is the 6 with a border.
I also fixed a typo in "rotation" in 1.
https://gist.github.com/maltelenz/098b8acaa0f079ff93aec93dcfd1187e

All look fine in SystemModeler:
Screenshot_20190830_121241

This is OpenModelica:
Screenshot_20190830_121158

To be honest, Dymolas rendering just looks broken, especially at the small sizes.

Edit: I'll make a bold statement: Broken tools should be ignored when writing a standard library for a language. We should use the originally suggested Joints icon. The majority of the tools had no trouble rendering it nicely, suggesting heavily that Dymola is just broken in this regard.

@beutlich
Copy link
Member Author

beutlich commented Sep 1, 2019

Screen shot from SimulationX:

grafik

@tobolar
Copy link
Contributor

tobolar commented Sep 2, 2019

Broken tools should be ignored when writing a standard library for a language.

In that case I would vote for Joints4 or Joints5. What I in particular preferre is: darker colors, larger circles representing the flange/bolt.

@maltelenz
Copy link
Contributor

I like Joints4 as well.

Copy link
Contributor

@tobolar tobolar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm in favor of reverting icon changes for Joints here and open another PR for it.

@beutlich beutlich merged commit 0a7a527 into modelica:master Sep 3, 2019
@beutlich beutlich deleted the better-icons branch September 3, 2019 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: Mechanics.MultiBody Issue addresses Modelica.Mechanics.MultiBody
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants