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

Update the names of files under docs/analog/model_parameters/LV/tables_clear #42

Open
mithro opened this issue Aug 2, 2022 · 2 comments · May be fixed by #51
Open

Update the names of files under docs/analog/model_parameters/LV/tables_clear #42

mithro opened this issue Aug 2, 2022 · 2 comments · May be fixed by #51
Assignees

Comments

@mithro
Copy link
Contributor

mithro commented Aug 2, 2022

The names of the files under docs/analog/model_parameters/LV/tables_clear should probably match primitive naming?

It might actually just be that the formatting of these file names need to match the other ones in this file? The files which made me think where the following;

  • docs/analog/model_parameters/LV/tables_clear/19_mos_3p3.csv
  • docs/analog/model_parameters/LV/tables_clear/21_mos_6p0.csv
  • docs/analog/model_parameters/LV/tables_clear/22_mos_6p0_1.csv
  • docs/analog/model_parameters/LV/tables_clear/21_mos_6p0_2.csv
@mithro
Copy link
Contributor Author

mithro commented Aug 2, 2022

I think they might just be _MOS_ or _Mos_ -- _Mos_ doesn't look right, so I'm guessing it _MOS_ is right?

@mohanad0mohamed mohanad0mohamed self-assigned this Aug 4, 2022
@mohanad0mohamed
Copy link
Contributor

Should we stick to the naming convetion of nfet/pfet instead?
I'm suggesting 19_mos_3p3.csv to be 19_nfet_pfet_03v3.csv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

2 participants