cesium-ml / cesium

Machine Learning Time-Series Platform
Other
670 stars 101 forks source link

Slightly robustify Featureset indexing #232

Closed bnaul closed 7 years ago

bnaul commented 7 years ago

@stefanv any thoughts on this __getitem__ method? I want to override the xr.Dataset indexing it's easy to get stuck in an infinite loop if you're not careful...this feels like a more robust solution to me, just operate on a pure Dataset version so you know you can't get back into a Featureset method by mistake.