[
https://issues.apache.org/jira/browse/CLIMATE-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14183751#comment-14183751
]
Kim Whitehall commented on CLIMATE-536:
---------------------------------------
Great discussions are going on re: this issue. I took a closer look at the
mccsearch.py code and realized that there is actually no dependency on files.py
It must have ended up in the folder from sloppy work on my part on the earlier
versions of mccsearch.py where I adapted the module readMergData(dirname,
filelist = None) from a module that existed in file.py
So we can go ahead and remove file.py from mccsearch/code.
I think however this discussion is useful for future applications and maybe
better addressed by opening a more appropriate ticket. What do you guys think?
> Update mccsearch files.py to use current replacement of utils.fortran function
> ------------------------------------------------------------------------------
>
> Key: CLIMATE-536
> URL: https://issues.apache.org/jira/browse/CLIMATE-536
> Project: Apache Open Climate Workbench
> Issue Type: Task
> Components: mcc search
> Affects Versions: 0.5
> Reporter: Lewis John McGibbney
> Assignee: Kim Whitehall
> Priority: Critical
> Fix For: 0.5
>
>
> Currently files.py within the mccsearch module uses a legacy (incorrect)
> fortranfile.py (0)... which hails from the now deprecated rcmet
> fortranfile.py (1)
> We need to make a determination as to how this should be upgraded.
> In particular, we use the following functions
> * [fortranfile.FortranFile(fileName,
> mode='w')|https://github.com/apache/climate/blob/master/mccsearch/code/files.py#L529]
> (0) https://github.com/apache/climate/blob/master/mccsearch/code/files.py#L32
> (1)
> https://github.com/apache/climate/blob/master/rcmet/src/main/python/rcmes/utils/fortranfile.py
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
