search

sdtaylor / phenology_dataset_study

1 stars 1 forks source link

do doy estimate better #7

Closed sdtaylor closed 7 years ago

sdtaylor commented 7 years ago

Potentially better (quicker) way to calculate doy_estimate in models.py. Must deal with #6 first.

    def doy_estimator2(self, gdd, doy_index, F):
        doy_index = doy_index.copy()
        doy_index = np.tile(doy_index, (gdd.shape[0],1))
        doy_index[gdd<F] = 10000
        return doy_index.argmin(1)
sdtaylor commented 7 years ago

I think the confusing code this creates isn't worth the benefit