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

PM documentation #3988

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

PM documentation #3988

wants to merge 9 commits into from

Conversation

AHaumer
Copy link
Contributor

@AHaumer AHaumer commented Jun 8, 2022

I recognized that modeling of permanent magnets in electrcal machines would need some explanations.

@AHaumer AHaumer added enhancement New feature or enhancement L: Magnetic.FundamentalWave Issue addresses Modelica.Magnetic.FundamentalWave documentation Issue addresses the documentation labels Jun 8, 2022
@AHaumer AHaumer self-assigned this Jun 8, 2022
Copy link
Member

@beutlich beutlich left a comment

Choose a reason for hiding this comment

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

Just a quick spelling check.

@AHaumer AHaumer enabled auto-merge June 9, 2022 05:03
@AHaumer AHaumer requested a review from beutlich June 9, 2022 05:03
@beutlich beutlich removed their request for review June 9, 2022 19:43
@beutlich beutlich changed the title PM documenation PM documentation Jun 9, 2022
This reverts commit ea740a58823b9ce65f6b54d94a1208dfe7159f98.
Copy link
Member

@dietmarw dietmarw left a comment

Choose a reason for hiding this comment

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

@AHaumer I had to revert your "???" commit once again. Please follow the instruction I gave you of basing your future PRs on the current master (rather than keeping to merge the current master into your "broken" local master with that "???" commit).
Otherwise, it looks good.

@beutlich beutlich disabled auto-merge June 13, 2022 08:19
@beutlich beutlich enabled auto-merge (squash) June 13, 2022 08:20
@AHaumer AHaumer requested a review from beutlich June 15, 2022 16:09
@beutlich beutlich removed their request for review September 24, 2022 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issue addresses the documentation enhancement New feature or enhancement L: Magnetic.FundamentalWave Issue addresses Modelica.Magnetic.FundamentalWave
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants