You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@climate.apache.org by agoodm <gi...@git.apache.org> on 2016/11/16 00:21:09 UTC

[GitHub] climate pull request #422: [WIP][CLIMATE-847] Support multiple observation /...

GitHub user agoodm opened a pull request:

    https://github.com/apache/climate/pull/422

    [WIP][CLIMATE-847] Support multiple observation / reference datasets in run_RCMES.py

    This PR will generalize RCMES to handle all datasets in the same way. Now rather than making any distinction from observation and model in the code, all datasets are loaded at once with the first dataset listed being designated as the reference for binary metrics. Theoretically, this capability was already possible but these changes should make it feel a bit more streamlined.
    
    ## Todos
    
    - [ ] Update configuration files in examples
    - [ ] Test an example configuration file with only observation / reanalysis datasets. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/agoodm/climate CLIMATE-847

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/climate/pull/422.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #422
    
----
commit 5971be65e956788dec3d65af98d6bac518df3fa6
Author: Alex Goodman <ag...@users.noreply.github.com>
Date:   2016-11-16T00:14:06Z

    Update run_RCMES.py to treat obs and model datasets the same

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] climate issue #422: [WIP][CLIMATE-847] Support multiple observation / refere...

Posted by OCWJenkins <gi...@git.apache.org>.

Github user OCWJenkins commented on the issue:

    https://github.com/apache/climate/pull/422
  
    Can one of the admins verify this patch?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] climate pull request #422: [CLIMATE-847] Support multiple observation / refe...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/climate/pull/422


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] climate issue #422: [WIP][CLIMATE-847] Support multiple observation / refere...

Posted by agoodm <gi...@git.apache.org>.

Github user agoodm commented on the issue:

    https://github.com/apache/climate/pull/422
  
    Alright, that should do it! @huikyole , please review and try out some tests with multiple observation datasets. I have just tested a few CORDEX-Africa examples and found them working just fine \U0001f44d .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---