Skip to content

Pid parameters updates (#509)#710

Merged
baggepinnen merged 2 commits intov1from
v1pid
Jul 6, 2022
Merged

Pid parameters updates (#509)#710
baggepinnen merged 2 commits intov1from
v1pid

Conversation

@baggepinnen
Copy link
Copy Markdown
Member

No description provided.

* stash

* pidparams suggestion

* pid creation update (ss) and s2f and f2s

* ss default, new filter

* remove paralleltime, inf->Inf

* update some tests

* single conversion method with param detection

* update errors

* Handle special cases

* Update error capitalization

* remove gof usage

* fix pidplots

* Fix pid creation, update stabregionpid

* Fix test

* Update docs

* Fix test pid in tf form

* Improve error for pid ss with D-part

* Update src/pid_design.jl

Co-authored-by: olof3 <olof3@users.noreply.github.com>

* stash

* new interface suggestion

* revert some code and make stuff work

* Ts=0 to Ts=nothing

* change parameter names when standard form

* kp/ki/kd -> param_p/param_i/param_d

* fix functions missing form

* fix tests

* fix errors to pass tests

* more tests

* Ts nothing

* make leadlinkcurve recipe and doplot on stabreg

* fix some docs

Co-authored-by: olof3 <olof3@users.noreply.github.com>
Co-authored-by: Fredrik Bagge Carlson <baggepinnen@gmail.com>
@JuliaControlBot
Copy link
Copy Markdown

Something failed when generating plots. See the log at https://github.com/JuliaControl/ControlExamplePlots.jl/actions/runs/2621124825?check_suite_focus=true for more details.

@baggepinnen baggepinnen merged commit 1d042ff into v1 Jul 6, 2022
@baggepinnen baggepinnen deleted the v1pid branch July 6, 2022 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants