ContinuumIO / elm

Phase I & part of Phase II of NASA SBIR - Parallel Machine Learning on Satellite Data
http://ensemble-learning-models.readthedocs.io
44 stars 23 forks source link

predicting VIC soil moisture based on NLDAS Forcing A #181

Closed PeterDSteinberg closed 7 years ago

PeterDSteinberg commented 7 years ago

cc @jbednar @gbrener @gpfreitas

PeterDSteinberg commented 7 years ago

TODO reminder for myself:

nldas_soil_moisture_ml.py:432:5: F841 local variable 'X_time_steps' is assigned to but never used
nldas_soil_moisture_ml.py:444:5: F841 local variable 'last_hr' is assigned to but never used
nldas_soil_moisture_ml.py:470:5: F841 local variable 'Xnew' is assigned to but never used
PeterDSteinberg commented 7 years ago

This screenshot shows the notebook in this PR:

screen shot 2017-07-11 at 8 07 28 am

Currently the script's ML approach to predicting the next hour's soil moisture has a 10% bias relative to VIC soil moisture. The two approaches have predictions with similar central tendencies, but Elm has more pos/neg outliers than VIC. I think that is a common problem in statistical vs deterministic models. I'll use today's demo to come up with improvement ideas.

gbrener commented 7 years ago

@PeterDSteinberg, the color scales now match up. I also equalized the histogram bin ranges to make comparisons a bit easier:

screen shot 2017-07-17 at 4 24 50 pm screen shot 2017-07-17 at 4 25 00 pm
PeterDSteinberg commented 7 years ago

Thanks ! That's a lot more useful now (the fix of the color scales), @gbrener . We should make sure the viz libraries' docs show this kind of thing directly in an easy to find way. It wasn't immediately obvious to me when googling around.

gbrener commented 7 years ago

Yeah, I actually needed @philippjfr's help to make it work (he designed much of the API). He acknowledged that the docs leave a bit to be desired in this area.