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

Parameter sweep issues #186

Closed
tomhajjar opened this issue Nov 22, 2022 · 5 comments · Fixed by #615
Closed

Parameter sweep issues #186

tomhajjar opened this issue Nov 22, 2022 · 5 comments · Fixed by #615
Labels

Comments

@tomhajjar
Copy link

tomhajjar commented Nov 22, 2022

I know we discussed this but I'm still seeing issues when doing Parameter sweeps of tubes and FETs. I think the issue may the same for any Parameter sweep using an even number of Points.

In this example below, I want plots of Vgrid=0 -> Vgrid=-max. If I use Start=-5, Stop =0, Step=1, Points=6, the first plot is for Vgs=-0.833 not Vgs=0.

To get the plot Vgs=0, I have to change Points to 5. For this setting it says Step is 1.25 when Step is actually 1.

2022-11-21_184906

2022-11-21_184734

Tube_6CA7_Push_Pull_prj.zip

@ra3xdh
Copy link
Owner

ra3xdh commented Nov 22, 2022

The Ngspice and Qucsator has different meaning of "step" and "intervals count" sweep. This issue is explained here #8

@tomhajjar
Copy link
Author

So Steps is wrong for ngspice and xyce spice but correct for Qucs. Since Qucsator is being depreciated, is it time to make a change or use the selection of simulator to change the definition?

@cmat21
Copy link

cmat21 commented Sep 26, 2023

It is somewhat confusing to have a difference in DC sweeps using the same settings. Is it planned to correct this difference?
dc sweep diference

@ra3xdh
Copy link
Owner

ra3xdh commented Sep 26, 2023

The DC sweep using DC+ParSweep was inherited from old Qucs. Qucsator has no DC sweep simulation and uses this combination instead. It also uses different method to calculate step. The same explantion as here: #8 I will think how to provide a fix and don't break the compatibility between simulators. It is not an obvious task.

@ra3xdh
Copy link
Owner

ra3xdh commented Mar 4, 2024

Fixed by #615

@ra3xdh ra3xdh closed this as completed Mar 4, 2024
@ra3xdh ra3xdh added the ngspice label Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants