Skogestad-Python icon indicating copy to clipboard operation
Skogestad-Python copied to clipboard

Remove tf class from utils

Open alchemyst opened this issue 8 years ago • 6 comments

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.

alchemyst avatar Feb 07 '18 14:02 alchemyst

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

DeonDataScience avatar Apr 23 '18 16:04 DeonDataScience

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

WillemASmit avatar Feb 26 '19 05:02 WillemASmit

@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.

alchemyst avatar Feb 26 '19 05:02 alchemyst

✅ Dead time feature request noted :)

ilayn avatar Feb 26 '19 06:02 ilayn

@ilayn You're too quick!

alchemyst avatar Feb 26 '19 06:02 alchemyst

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.

DeonDataScience avatar Feb 26 '19 09:02 DeonDataScience