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

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

Github user cclauss commented on a diff in the pull request:

    https://github.com/apache/climate/pull/439#discussion_r131525571
  
    --- Diff: mccsearch/code/mainProgTemplate.py ---
    @@ -59,36 +59,36 @@ def main():
         # 
-------------------------------------------------------------------------------------------------
     
         # let's go!
    -    print "\n -------------- Read MERG Data ----------"
    +    print("\n -------------- Read MERG Data ----------")
         mergImgs, timeList = mccSearch.readMergData(CEoriDirName)
    -    print ("-" * 80)
    +    print(("-" * 80))
    --- End diff --
    
    Recommendation to change `print(("-" * 80))` to `print("-" * 80)` in eight 
places.  The double nested parens `(( ))` are not needed and only slow down 
your reader.  Repeated on lines: 64, 75, 77, 80, 82, 86, 89, 112.


> Upgrade mccSearch code from Python2 > 3
> ---------------------------------------
>
>                 Key: CLIMATE-912
>                 URL: https://issues.apache.org/jira/browse/CLIMATE-912
>             Project: Apache Open Climate Workbench
>          Issue Type: Improvement
>          Components: mcc search
>            Reporter: Lewis John McGibbney
>            Assignee: Lewis John McGibbney
>            Priority: Minor
>             Fix For: 1.3.0
>
>
> I recently attempted to run the MCC Search code without any luck. Thsi issue 
> will simply upgrade the code such that it runs with Python 3.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to