Refactor IndexableDataset constructor

mila-iqia / fuel

A data pipeline framework for machine learning

MIT License

867 stars 268 forks source link

Refactor IndexableDataset constructor #148

Open dwf opened 9 years ago

dwf commented 9 years ago

Right now it's somewhat hard to write a class that inherits from IndexableDataset and doesn't duplicate a bit of annoying logic. IIRC some inheriting classes have forgone start and stop for a subset argument, should deal with that as well.

vdumoulin commented 8 years ago

@dwf is this issue still relevant? If so, could you elaborate?