Move data reader out of Temporal
femtotrader opened this issue · 2 comments
Quandl, Google Finance, and Yahoo Finance data readers are also concerned
@femtotrader I don't necessarily disagree with you on this point from a best practices standpoint. The reason I originally undertook the effort to do all this was really that Temporal was a package that was not very widely used, and as such there are more difficulties maintaining this support in other more general packages.
Currently I still feel that Temporal isn't popular enough that maintaining support for the TS type would be any kind of major concern for other package developers. This is my personal assessment, feel free to disagree. I also don't see much of a disadvantage in having it supported within this package -- might it be causing some headaches that I'm not aware of?