combine data source and data readers into single data module

BreakingBytes / simkit

Model Simulation Framework

http://breakingbytes.github.io/simkit/

BSD 3-Clause "New" or "Revised" License

27 stars 16 forks source link

Closed mikofski closed 8 years ago

mikofski commented 8 years ago

there is only going to be one data source in carousel core, so why does it need it's own module?

mikofski commented 8 years ago

closing this as won't fix:

there is already over 300 lines in data sources, so it's already long enough, so should not add new classes that are not directly related to data source
and there will be many readers, new readers are already in contrib, so it makes sense to keep readers separate and not combine them