ceholden / yatsm

Yet Another Time Series Model
https://yatsm.readthedocs.org/en/latest/
MIT License
63 stars 30 forks source link

Fix indexing of the dataset #99

Closed wkearn closed 7 years ago

wkearn commented 7 years ago

ds in data returns the name of the dataset (i.e. 'Landsat') which is a string and doesn't have an attrs attribute. This causes an error. It's an easy fix though to index into the data by this string.

wkearn commented 7 years ago

With this morning's fixes, I can run through my pipeline!

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 54.723% when pulling 27726dfc1daef160434c47171f9d54b407257dcf on wkearn:merge_attrs into d7f90f859153dfc76fc744928261611495158e97 on ceholden:master.