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

Add torque-control mode, refactoring in the simulation and mechanics modules #133

Merged
merged 1 commit into from
Jun 9, 2024

Conversation

mhinkkan
Copy link
Member

@mhinkkan mhinkkan commented Jun 9, 2024

This pull request contains the following changes:

  • An unnecessary set_initial_values() method from the simulation module has been removed.
  • The time values corresponding to the solution are now saved to all subsystem. This simplifies post-processing the data after the simulation in some cases (if the class uses functions of the simulation time).
  • The ExternalRotorSpeed class has been added to the mechanics module. This allows to simulate drive systems in torque-control mode. One example simulation script has also been added.
  • The parameters of the TwoMassMechanicalSystem class are now provided through the TwoMassMechanicalSystemPars dataclass.
  • The complex exponent is used an internal state for rotating angles in system models. This allows to avoid discontinuities originating from the angle wrapping, which could otherwise cause problems to the solver.
  • Notation for the external load torque has been simplified. The speed-dependent load torque term is now parameterized through the speed-dependent friction coefficient (can be constant or callable).

…anics module, use complex exponential as an internal state for the angles to avoid discontinuities due to wrapping, remove an unnecessary method from the simulation module.
@mhinkkan mhinkkan merged commit b5eeb05 into Aalto-Electric-Drives:main Jun 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant