JuliaControl / JuliaControl/ControlSystems.jl
Known Bugs
Open
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 582
- Forks
- 92
- Avg merge
- 1h 12m
- Merged PRs (30d)
- 2
Description
We can use this issue to keep track of known bugs, issues and the progress we are making after pulling the major autodiff branch
Bugs
-
polefor mimo system, should not return duplicates, see https://github.com/JuliaControl/ControlSystems.jl/issues/81 (PR merged #165 ) -
dampreportfails on complex poles -
minrealwill throw error for Integer systems if poles are not exact integers see https://github.com/JuliaControl/ControlSystems.jl/issues/127 -
feedback2dofsilently breaks for discrete systems see https://github.com/JuliaControl/ControlSystems.jl/issues/95 - call on discrete system with freq vector ignores "map_to_unit_circle" Fixed:
- Systems can have negative sample time (used for "unspecified sample time"), makes iscontinuous true, results in bugs in evalfr
Type problems
-
c2dofTransferFunction{SisoZpk}returns ``TransferFunction{SisoRational}`
Other issues
- Tests for plots are deactivated for now
-
c2dnot working for improper transfer functions, see https://github.com/JuliaControl/ControlSystems.jl/issues/128 - Multiple "TODO" in code needs to be be checked
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by choosing one unchecked item in the Bugs, Type problems, or Other issues lists and follow any referenced issues, such as #127, #95, or #128. The work is done when that specific problem is resolved and its checklist entry can be updated; this issue does not identify files or tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100