-
Notifications
You must be signed in to change notification settings - Fork 439
Maintenance #162
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
Comments
It would be great to get some help in cleaning things up. As you can see from the backlog of pull requests, we have a bit of a bottleneck. But I hope to spend some time cleaning things up between now and Jan 2018. In terms of backward compatibility: several people (including me) use this package for teaching and so backwards compatibility (eg, with Python 2.7) is useful. |
Ok cool, but when I said breaking backwards compatibility I didn't mean python, I meant reorganizing and renaming some bits. For example, statefbk has a "ctrb" function and your StateSpace class just takes "*args" and then you unpack that to at most five values. Anyway, do you have another channel of communication, besides the issue tracker and mailinglist? |
General rules for compatibility: if an example or unit test breaks, it probably requires some discussion since people may be using that feature. For things in the Current channels for communication are github and the mailing list, with the issue tracker preferred. |
That's easy enough, I'll just keep the pull requests small and manageable. |
Hi everyone,
I wanted to start a little project involving control theory today and it seems this is the python module to use.
Can I help you clean it up a bit? Is there someone to accept pull requests? Would you even accept one that breaks backward compatibility, considering this is probably used in scientific contexts?
The text was updated successfully, but these errors were encountered: