Skip to content
This repository has been archived by the owner on Jun 14, 2019. It is now read-only.

Update TSTools to reflect changes in YATSM Long Term Mean Phenology #62

Closed
valpasq opened this issue Nov 17, 2015 · 2 comments
Closed

Comments

@valpasq
Copy link

valpasq commented Nov 17, 2015

Getting this error when attempting to run the phenology calculations live with TSTools:

File "/usr3/graduate/valpasq/.qgis2/python/plugins/tstools/ts_driver/timeseries_yatsm.py", line 160, in fetch_results
    self._fetch_results_live()
  File "/usr3/graduate/valpasq/.qgis2/python/plugins/tstools/ts_driver/timeseries_yatsm.py", line 445, in _fetch_results_live
    ltm = pheno.LongTermMeanPhenology(self.yatsm_model)
AttributeError: 'module' object has no attribute 'LongTermMeanPhenology'

Figuring this has to do with the changes made to the YATSM phenology calculation the other week.

@ceholden
Copy link
Owner

You're correct -- I changed the API in YATSM and need to update this driver to accommodate. I've waited a bit for some good ideas to come along for how to manage versions between this TS driver and YATSM, but none have crept in so I'll just implement a simple version checking error as I had in mind.

A larger question is how to communicate driver specific changes to the user so one can know if anything relevant to their TS driver is changing. Sounds like I need another repository...

@ceholden ceholden added this to the 1.0.1 milestone Nov 19, 2015
ceholden added a commit that referenced this issue Nov 25, 2015
@ceholden
Copy link
Owner

Fixed it for me -- going to close this so I can release v1.0.1, but let me know if it's not working for you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants