-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11743/
-----------------------------------------------------------

(Updated June 10, 2013, 5:49 p.m.)


Review request for Apache Open Climate.


Changes
-------

Kyo posted a comment on JIRA suggesting that I use the resize() method instead 
of directly changing the shape since it would not affect performance and is 
easier for some people to understand.


Description
-------

In preparation for the performance tweaks I will make to metrics.py, I have 
added a helper function called reshapeMonthlyData() to rcmes/utils/misc.py that 
does the reshaping trick I mentioned earlier for monthly data. I also made it 
work for arrays with different dimensions (eg 1d and 3d) which will should help 
clean up the code considerably. 

An example of how it would be used has been attached in benchmark_metrics.py 


Diffs (updated)
-----

  
http://svn.apache.org/repos/asf/incubator/climate/trunk/rcmet/src/main/python/rcmes/utils/misc.py
 1490884 

Diff: https://reviews.apache.org/r/11743/diff/


Testing
-------

-Works for arrays with arbitrary dimensions as long as the first dimension is a 
multiple of 12
-Maintains original shape of input array


Thanks,

Alex Goodman

Reply via email to