Simon Filhol

Results 34 comments of Simon Filhol

actually there are more: ``` ./TopoPyScale/topo_utils.py:23:38: F821 undefined name 'resample_func' df = df.resample(freq).apply(resample_func) ^ ./TopoPyScale/topo_utils.py:39:38: F821 undefined name 'resample_func' df = df.resample(freq).apply(resample_func) ^ ./TopoPyScale/topo_utils.py:218:5: F821 undefined name 'FsmPlot' FsmPlot(df) ^...

and many hard path in function: `def val_plots():`

good point Joel. I ran into this behavior that is frustrating, indeed. We could instead have an overwrite flag in the `compute_dem_parameter()`. Would that be a good alternative?

Good point. What about if we add a flag in the YML file called `interactive_shell: True`? Simon

We can simply have an if statement checking for that flag right before the ìnput()`command. But then if it does not find the files it automatically downloads. Sounds good?

Implemented. See commits: - https://github.com/ArcticSnow/TopoPyScale/commit/988243973f699bdc965443cbc06eaeab544a4278 - https://github.com/ArcticSnow/TopoPyScale/commit/b157c1a85ee8da293494528c2444615b4f434837

There is a problem in seasonality of the rain vs snow precip.

The multiprocessing library has both multithread and multicore. one core can handle multithreads. It is very convenient for instance to send and handle the download request (requiring little computation). Maybe...

what is the colorscale on your plot? And I suspect those are DEM artefacts?

This is indeed strange! Could it be to bring aspect in the same convention as solar geom or Horizon which assumes the south at 0, west in negative and east...