nicolas998 / WMF

Watershed Modelling Framework
GNU General Public License v3.0
29 stars 20 forks source link

Correciones en la funcion plot_tc() para poder funcionar en python3 y en run_shia para poder compilar al instalar el develop #87

Closed seospinale closed 5 years ago

seospinale commented 5 years ago

La función plot_tc() había dejado de funcionar por los cambios en los métodos .values() y .keys() de los diccionarios, además en el último push ya no se podía compilar al instalar el branch develop debido a un error en la función run_shia al definir que se van a guardar los almacenamientos por lo siguiente:

    if models.save_storage == 1:
        rutaStorageHdr = __Add_hdr_bin_2route__(ruta_storage)
        #Caso en el que se registra el alm medio
        if models.show_storage == 1:
            __Save_storage_hdr__(ruta_sto_hdr,rain_ruteHdr,N_intervals,
                start_point,self,Mean_Storage = np.copy(models.mean_storage),
                WhereItSaves)
        #Caso en el que no hay alm medio para cada uno de los
        else:
            __Save_storage_hdr__(ruta_sto_hdr,rain_ruteHdr,N_intervals,
                start_point,self,Mean_Storage=np.zeros((5,N))*-9999,
                WhereItSaves)<-- como whereitsaves está después de un argumento asignado como arg=valor whereitsaves debe ser asignado de la misma manera

revisando el código parece que la solución es simplemente

    if models.save_storage == 1:
        rutaStorageHdr = __Add_hdr_bin_2route__(ruta_storage)
        #Caso en el que se registra el alm medio
        if models.show_storage == 1:
            __Save_storage_hdr__(ruta_sto_hdr,rain_ruteHdr,N_intervals,
                start_point,self,Mean_Storage = np.copy(models.mean_storage),
                WhereItSaves = WhereItSaves)
        #Caso en el que no hay alm medio para cada uno de los
        else:
            __Save_storage_hdr__(ruta_sto_hdr,rain_ruteHdr,N_intervals,
                start_point,self,Mean_Storage=np.zeros((5,N))*-9999,
                WhereItSaves = WhereItSaves) <--- agregar aqui = whereitsaves porque con ese nombre se definió previamente