-
Notifications
You must be signed in to change notification settings - Fork 52
⚡ Fossum model for carrier lifetime #2231
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
Conversation
8a0d539
to
8099b78
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
e5e543c
to
1b5c3cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but for my understanding where exactly do the specific values come from in the two separate mobility models now?
The Fossum parameters can be found in the original paper by Fossum et al. The mobility model is basically the same model as before but by splitting up the entry into n and p we avoid having to duplicate naming (e.g., mu_n, mu_p) I'll add some references. |
@momchil-flex @daquinteroflex if you both are OK with this I'll merge by the end of the day |
Makes sense to me! |
b9bc27c
to
72de779
Compare
72de779
to
c29bfc3
Compare
CaugheyThomasMobility
)This is the only thing that needed to be added to the front-end to improve the 2D results.
FYI: There will soon be a back-end PR associated to this plus some other fixes