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

Remove tf class from utils #307

Open
alchemyst opened this issue Feb 7, 2018 · 6 comments
Open

Remove tf class from utils #307

alchemyst opened this issue Feb 7, 2018 · 6 comments
Labels

Comments

@alchemyst
Copy link
Owner

The tf class is perhaps reaching the end of its usefulness. We don't really need to maintain a whole separate class from the ones found in control and harold or sympy.

@DeonDataScience
Copy link
Contributor

This does not seem that difficult to accomplish. I will work on this.

@WillemASmit
Copy link
Contributor

@DeonPMC how far did you get with this issue? @alchemyst inherently the mimotf class will also have to be removed?

@alchemyst
Copy link
Owner Author

@WillemASmit I am actually torn on this as right now neither of the linked libraries have the ability to handle dead time. This is a major stumbling block, so closing this issue probably will also require contributing support for delays to one of the other libraries.

@ilayn
Copy link

ilayn commented Feb 26, 2019

✅ Dead time feature request noted :)

@alchemyst
Copy link
Owner Author

@ilayn You're too quick!

@DeonDataScience
Copy link
Contributor

I never got around to changing this. In my opinion there is little reason to change the tf and mimotf classes. The greatest part of the code-base is dependent on these objects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants