[ 
https://issues.apache.org/jira/browse/CLIMATE-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14005658#comment-14005658
 ] 

ASF GitHub Bot commented on CLIMATE-349:
----------------------------------------

Github user agoodm commented on the pull request:

    https://github.com/apache/climate/pull/37#issuecomment-43852675
  
    I agree with Cam's sentiment, we should never add extra dependencies just 
to make it a little bit easier to write unit tests. That said, I am fairly sure 
that dateutil is a required dependency for matplotlib, which is itself needed 
to generate all of the plots in OCW. I would assume that anyone that is capable 
of making plots with OCW would also have dateutil installed, which is why I am 
guessing Mazi didn't realize that he already had it installed.
    
    Cam, can you successfully import matplotlib on your machine but not 
dateutil?


> Refactoring "reshapeMonthlyData" from rcmes/utils/misc.py 
> ----------------------------------------------------------
>
>                 Key: CLIMATE-349
>                 URL: https://issues.apache.org/jira/browse/CLIMATE-349
>             Project: Apache Open Climate Workbench
>          Issue Type: Improvement
>          Components: metrics
>    Affects Versions: 0.3-incubating
>            Reporter: Maziyar Boustani
>            Assignee: Maziyar Boustani
>             Fix For: 0.4
>
>
> Refactoring function "reshapeMonthlyData" from [1] to [2] and providing 
> unittest for that.
> [1]:https://svn.apache.org/repos/asf/incubator/climate/trunk/rcmet/src/main/python/rcmes/utils/misc.py
> [2]:https://svn.apache.org/repos/asf/incubator/climate/trunk/ocw/utils.py



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to