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

Deprecate SweepModel property of Parameter Sweep. #839

Closed
ra3xdh opened this issue Jul 12, 2024 · 1 comment · Fixed by #838
Closed

Deprecate SweepModel property of Parameter Sweep. #839

ra3xdh opened this issue Jul 12, 2024 · 1 comment · Fixed by #838
Assignees
Labels
Milestone

Comments

@ra3xdh
Copy link
Owner

ra3xdh commented Jul 12, 2024

The SweepModel property was introduced to sweep model parameters using @device[param] syntax. The recent Ngspice versions has introduce alterparam directive that allows to sweep variables. The functionality of the SweepModel property could be implemented using the variable sweep and explicit specification is not needed anymore. I propose to deprecate these features, because it can mislead user. The variable weep is also unified with Qucsator syntax.

@ra3xdh ra3xdh added the ngspice label Jul 12, 2024
@ra3xdh ra3xdh self-assigned this Jul 12, 2024
@ra3xdh ra3xdh linked a pull request Jul 12, 2024 that will close this issue
@ra3xdh
Copy link
Owner Author

ra3xdh commented Jul 12, 2024

I have implemented this proposal as a part of #838

@ra3xdh ra3xdh added this to the 24.3.0 milestone Jul 16, 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.

1 participant