----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12072/ -----------------------------------------------------------
(Updated June 26, 2013, 3:56 p.m.) Review request for Apache Open Climate, Cameron Goodale, Chris Mattmann, Michael Joyce, and Paul Ramirez. Changes ------- The second version of dataset.py has been modified based on Cam's and Mike's comments. The getTemporalResolution function improved. The format of documentation has been changed. Mazi Repository: climate Description ------- Based on the new structure of OCW (Open Climate Workbench) [1], the dataset.py is the critical file that will contain all the functions and attributes related to any type of dataset (local observations, model file, RCMED and esg). This is the first time the dataset.py is developed, therefore it may need to be modified more as we go ahead. [1]: https://cwiki.apache.org/confluence/display/CLIMATE/Open+Climate+Workbench+API+summary [STRUCTURE A] Diffs (updated) ----- https://svn.apache.org/repos/asf/incubator/climate/trunk/dataset.py PRE-CREATION Diff: https://reviews.apache.org/r/12072/diff/ Testing ------- There are three files attached to this review board that can provide the test situation for you. 1- local.py ---> This file is not the final version of local.py. However, it is a good source to test the dataset.py but a model file[3]. 2- test.py ---> This file will use local.py[1] and model file[3] to test all the attributes from dataset.py. 3- prec.CRCM.ncep.monavg.nc ---> This is the actual model file that can be used to test the dataset.py (feel free to use your model file and report if any issue found) File Attachments ---------------- This is not the final version of local.py. However, the dataset.py can be test by this local.py (code example in "Testing area"). https://reviews.apache.org/media/uploaded/files/2013/06/25/local.py This https://reviews.apache.org/media/uploaded/files/2013/06/25/test.py dfd https://reviews.apache.org/media/uploaded/files/2013/06/25/prec.CRCM.ncep.monavg.nc Thanks, Maziyar Boustani
