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

Wrong parameter in Magnetic/FluxTubes/Material #4537

Open
AHaumer opened this issue Feb 19, 2025 · 5 comments
Open

Wrong parameter in Magnetic/FluxTubes/Material #4537

AHaumer opened this issue Feb 19, 2025 · 5 comments
Assignees
Labels
bug Critical/severe issue L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes
Milestone

Comments

@AHaumer
Copy link
Contributor

AHaumer commented Feb 19, 2025

The parameter sigma = 3.22e6 in Magnetic/FluxTubes/Material/HysteresisTableData/M400_50A seems to be wrong,
compared with sigma = 2.22e6 in Magnetic/FluxTubes/Material/HysteresisEverettParameter/M400_50A.
Thanks to Hans Hell for this report!
sigma is the Electrical conductivity of material [S/m].
This value varies a lot with silicon doping. I'll try to search for parameters given by steel manufacturers.
@casella should we try to fix this for 4.1.0?

@AHaumer AHaumer added bug Critical/severe issue L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes labels Feb 19, 2025
@AHaumer AHaumer added this to the MSL4.1.0 milestone Feb 19, 2025
@AHaumer
Copy link
Contributor Author

AHaumer commented Feb 19, 2025

I found https://www.emt-net.de/data/uploads/motorkomponenten/allgemeine-informationen-zu-elektroblechen.pdf:
Spezifischer Widerstand = Resistance = 0.48μΩm which leads to sigma = 2.08e6 S/m
Therefore the value in TableData seems to be a typo.

@casella
Copy link
Contributor

casella commented Feb 24, 2025

We could do it, the question is how many consequences does this have. Is it affecting any reference results? Do we need to report this in the release notes?

You can start by opening a PR on master, then we can decide whether to put it in 4.1.0 or not.

@AHaumer
Copy link
Contributor Author

AHaumer commented Feb 25, 2025

I'll create a PR today's evening, also suggesting a short text for the release notes.
No this material M400_50A (there are 8 different material data sets in FluxTubes.Material.HysteresisTableData) is not used in an example or in Modelica.Test, therefore no reference results are affected,

@AHaumer
Copy link
Contributor Author

AHaumer commented Feb 25, 2025

See PR #4544

@casella
Copy link
Contributor

casella commented Feb 25, 2025

@Esther-Devakirubai this will need to be back-ported to maint/4.1.x and a note added to the release notes about model fixes.

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

No branches or pull requests

2 participants