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.

Sign up for GitHub

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

Jump to bottom

Units for gamma in Modelica.Electrical.Spice3.Internal.Mos.mos2CalcCalcTempDependenciesRevised #4369

Open
HansOlsson opened this issue Mar 27, 2024 · 0 comments · May be fixed by #4377
Open

Units for gamma in Modelica.Electrical.Spice3.Internal.Mos.mos2CalcCalcTempDependenciesRevised #4369

HansOlsson opened this issue Mar 27, 2024 · 0 comments · May be fixed by #4377
Labels
L: Electrical.Spice3 Issue addresses Modelica.Electrical.Spice3 requires Modelica 3.7 Issue that requires Modelica Language Specification 3.7

Comments

@HansOlsson
Copy link
Contributor

Modelica.Electrical.Spice3.Internal.Mos.mos2CalcCalcTempDependenciesRevised contains:

out_c.m_tVbi := in_p.m_vt0 - in_m_type*(in_p.m_gamma*sqrt(in_p.m_phi)) + ...
The in_m_type is just an integer, but the other variables are all voltages, which mean that we have voltage=voltage+voltage*sqrt(voltage), which clearly does not make sense.

Looking at references it seems that the formula looks ok, but gamma is computed as the square root of something, and thus likely should have unit sqrt(voltage) instead (which will require Modelica Language 3.7).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
L: Electrical.Spice3 Issue addresses Modelica.Electrical.Spice3 requires Modelica 3.7 Issue that requires Modelica Language Specification 3.7
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Proper unit for gamma. It is computed as the square root of something. HansOlsson/Modelica
1 participant
@HansOlsson