Johan Euphrosine

Results 573 comments of Johan Euphrosine

I'm back from leave and should be able to make progress on this! Here is my plan based on the past discussions on this: - [x] Reproduce the plot from...

@RTimothyEdwards please have a look at https://github.com/google/skywater-pdk/issues/381#issuecomment-1237564571, the continuous models don't seems to fix the continuity issue reported in #381. Should we still move forward with the the remaining items...

@kgugala @PiotrZierhoffer found #143 #256 and #266 that all seem to contribute to fixing this, could you advise on the best way to merge them (rebase, apply in order, drop...

http://www2.hawaii.edu/~whitece6/posts/firstpost/ actually has a testbench + a notebook that reproduce the issue, we should test that the model in #380 actually fixes the glitch.

@hpretl trying to reproduce the testbench linked in http://www2.hawaii.edu/~whitece6/posts/firstpost/ with `ngspice` + the new models from #380 ``` nfet1V8 I-V curve .param mc_switch=0 .lib ../../ngspice/sky130.lib.spice tt * Gate bias X1...

Removing the `u` suffix from: ``` X1 vdn vgn 0 vbn sky130_fd_pr__nfet_01v8 w=10 l=0.15 ``` seems to workaround the error in https://github.com/google/skywater-pdk/issues/381#issuecomment-1236656126 but it doesn't seems that `ngspice` save anything...

tried to reproduce the plot in the original submission from @hpretl: https://github.com/google/skywater-pdk/issues/381#issue-1294635084 without much success. # sky130_fd_pr__nfet_01v8 ## binned ``` sky130_fd_pr__nfet_01v8 gm/id curve .param mc_switch=0 .lib ../../../conda-env/share/pdk/sky130A/libs.tech/ngspice/sky130.lib.spice tt * Gate...

@bmurmann yep, but I wanted to give @hpretl original test bench a try before :)

tried w/ the pfet as pointed by https://github.com/google/skywater-pdk/issues/381#issuecomment-1237314806 and http://www2.hawaii.edu/~whitece6/posts/firstpost/#testing-with-the-pmos and it doesn't seems that it fixes the glitch. # sky130_fd_pr__pfet_01v8 ## binned ``` sky130_fd_pr__pfet_01v8 gm/id binned models .param mc_switch=0...