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

Varistor library doesn't work....anymore?? #548

Closed
tomhajjar opened this issue Feb 10, 2024 · 6 comments · Fixed by #555
Closed

Varistor library doesn't work....anymore?? #548

tomhajjar opened this issue Feb 10, 2024 · 6 comments · Fixed by #555

Comments

@tomhajjar
Copy link

tomhajjar commented Feb 10, 2024

I haven't used the Varistor library in a long time but when making a test folder of DC sweeping devices, the library didn't work.

I know it worked in the past #92 (2022) but unsure of the details. I recovered the data from an old backup.

The Curve Tracer project might be useful for new users.

2024-02-10_102544

2024-02-10_110514

2022 backup.zip

Curve Tracer Testing_prj.zip

@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

This library doesn't work anymore when using Ngspice-39. If the Ngspice version <= 37 everything works as expected.

@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

The qucs2spice::convert_rcl() need to be modified. Ngspice requires to wrap variable value in braces to distinguish it from model.

But the best solution will be to remake the Varistor.lib to provide true SPICE netlist. This task cannot be done in the near future, because the library contains about 100 devices.

@tomhajjar
Copy link
Author

I will look into available models.

@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

I will try to provide a fix of qucs2spice.cpp in the next few days.

@tomhajjar
Copy link
Author

tomhajjar commented Feb 12, 2024

The Epcos/TDK, varistor library is for "block" style varistors which have a much higher current rating than the generic 20mm leaded disk varistors used in most products. Not really an issue for varistor simulations where voltage is the primary parameter other than the parasitics being higher. Not sure why this type of varistor was used.

s-l1600

SIOV_Block_HighE.zip

@ra3xdh ra3xdh linked a pull request Feb 12, 2024 that will close this issue
@ra3xdh
Copy link
Owner

ra3xdh commented Feb 12, 2024

Fixed via #555 Varistor library is operational again. Application rebuild is required to apply the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants