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

Bug in Modelica.Mechanics.Rotational.Gearbox #381

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 4 comments
Closed

Bug in Modelica.Mechanics.Rotational.Gearbox #381

modelica-trac-importer opened this issue Jan 14, 2017 · 4 comments
Assignees
Labels
bug Critical/severe issue invalid Invalid issue L: Mechanics.Rotational Issue addresses Modelica.Mechanics.Rotational
Milestone

Comments

@modelica-trac-importer
Copy link

Reported by stefanv on 11 Jun 2010 13:54 UTC
Modelica.Mechanics.Rotational.Gearbox includes a LossyGear, which has useSupport=true. The Gear's support is connected to the Gearbox's support. However, the Gearbox inherits its support from Interfaces.PartialTwoFlangesAndSupport, which by default has useSupport=false. The extends clause does not modify this, but it should.

In short, the extends clause of Modelica.Mechanics.Rotational.Gearbox should read:

extends Modelica.Mechanics.Rotational.Interfaces.PartialTwoFlangesAndSupport(useSupport=true);

Can someone with the appropriate privileges please correct this (in MSL 3.1 and 3.2)?


Migrated-From: https://trac.modelica.org/Modelica/ticket/381

@modelica-trac-importer
Copy link
Author

Comment by stefanv on 11 Jun 2010 18:36 UTC
After some discussion with my colleagues, I believe I might have been wrong. We still think there is a bug, but that the correct fix is to change the declaration of LossyGear within the model:

  Rotational.Components.LossyGear lossyGear(**final** ratio=ratio, **final** lossTable=lossTable, **final** useSupport=useSupport) annotation ...

PS. The model should have been Modelica.Mechanics.Rotational.Components.Gearbox (I left out "Components" earlier).

@modelica-trac-importer
Copy link
Author

Modified by dietmarw on 11 Jun 2010 19:41 UTC

@modelica-trac-importer modelica-trac-importer added the L: Mechanics.Rotational Issue addresses Modelica.Mechanics.Rotational label Jan 14, 2017
@modelica-trac-importer modelica-trac-importer added this to the MSL3.2 milestone Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Comment by AHaumer on 8 Jul 2010 08:35 UTC
Sorry Stefan you're wrong:
GearBox contains LossyGear. LossyGear.useSupport is set final to true, since LossyGear's conditional support (which isn't accessible from outside of the GearBox) is connected to the internal support of the GearBox. The user can choose to use the conditional external support of the GearBox or not. The support torque will show up either at the conditional interal "fixed" of the GearBox (GearBox.useSupport=false) or at the external support (GearBox.useSUpport=true).
I hope this clarifies the usage of conditional support (and conditional HeatPort).

Unfortunately I can't close the ticket (won't fix) - why? (See ticket #382).

@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 8 Jul 2010 15:14 UTC
So actually this means that the ticket is invalid rather than "Won't fix".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue invalid Invalid issue L: Mechanics.Rotational Issue addresses Modelica.Mechanics.Rotational
Projects
None yet
Development

No branches or pull requests

2 participants