You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@climate.apache.org by MJJoyce <gi...@git.apache.org> on 2015/02/10 00:35:51 UTC

[GitHub] climate pull request: CLIMATE-575 - Initial config based evaluatio...

GitHub user MJJoyce opened a pull request:

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

    CLIMATE-575 - Initial config based evaluation runner implementation

    - Implements initial config based execution of evaluations. There are a number of missing features, the most obvious of which is the lack of support for all the available plot types in OCW. These and other things will be updated in tickets that follow.
    - Included is a config for one of our example files (simple_model_to_model_bias). The evaluation area is   slightly changed, but other than that all is the same. It is assumed that the user has the files downloaded to /tmp when using this config file.

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

    $ git pull https://github.com/MJJoyce/climate CLIMATE-575

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

    https://github.com/apache/climate/pull/152.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 #152
    
----
commit c958da0979c13539ad1bf43c900d075de88e1b6f
Author: Michael Joyce <jo...@apache.org>
Date:   2015-02-09T22:58:27Z

    CLIMATE-575 - Initial config based execution implementation
    
    - Implement initial config based execution of evaluations. There are a
      number of missing features, the most obvious of which is the lack of
      support for all the available plot types in OCW. These and other
      things will be updated in tickets that follow.

commit 4c9abe7bd2de834d51ad23747f3ef7fd3a7630b1
Author: Michael Joyce <jo...@apache.org>
Date:   2015-02-09T23:33:00Z

    CLIMATE-575 - Test fixes and improvements
    
    - Fix bug in valid_plot_config helper that was left in from a copy-paste
      error.
    - Switch all yaml.safe_load calls to yaml.load to mirror the
      implementation.
    - Add tests for the contour map validation that is currently
      implemented.
    - Add tests for the generic plot config validation errors.
    - Add an additional check for the config_is_well_formed helper to ensure
      that invalid plot data is reported correctly.

----


---
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: CLIMATE-575 - Initial config based evaluatio...

Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:

    https://github.com/apache/climate/pull/152#issuecomment-73615004
  
    Merged build finished. Test Passed.


---
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: CLIMATE-575 - Initial config based evaluatio...

Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:

    https://github.com/apache/climate/pull/152#issuecomment-73614844
  
     Merged build triggered. Test Failed.


---
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: CLIMATE-575 - Initial config based evaluatio...

Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:

    https://github.com/apache/climate/pull/152#issuecomment-73614853
  
    Merged build started. Test Failed.


---
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: CLIMATE-575 - Initial config based evaluatio...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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.
---