Repository navigation
Document how values are computed based on resonance frequency. - #4780
HansOlsson wants to merge 3 commits into
Conversation
Also makes the models unit correct.
|
That goes into the right direction. Sorry that it got stuck for such a long time. |
Good. I understood the concept, but I didn't know the specific name for it in the electrical domain. |
Suggested by A. Haumer Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>
|
Sorry there was a bug in the pdf: Pls. don't use "R" as characteristic impedance, "R" is a resistance. |
|
We could take the following code snippet as a reference (also for other PRs):
|
It also makes the models unit correct, but the main goal is to make it easier to explain the models - by replacing literal values with unclear relations with somewhat understandable parameters, and clear formulas.
(It could be made even better by explaining the derivation of them.)
A benefit is that it is straightforward to modify the resonance frequency (or resistance.) and re-run the simulation to see the impact. (Based on similar idea in #4774 )
In terms of units more work is needed - there are more models that don't work well if pi have unit "rad" or "1"; all indicating unit issues e.g.,:
Does not require Modelica Language version 3.7 or updated pi-definition.