Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

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

Merged
merged 3 commits into from Nov 28, 2016

Conversation

agoodm
Copy link
Member

@agoodm agoodm commented Nov 16, 2016

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.

@OCWJenkins
Copy link

Can one of the admins verify this patch?

@agoodm
Copy link
Member Author

agoodm commented Nov 16, 2016

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

@agoodm agoodm changed the title [WIP][CLIMATE-847] Support multiple observation / reference datasets in run_RCMES.py [CLIMATE-847] Support multiple observation / reference datasets in run_RCMES.py Nov 16, 2016
@asfgit asfgit merged commit 7187cf3 into apache:master Nov 28, 2016
@agoodm agoodm deleted the CLIMATE-847 branch November 28, 2016 22:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants