Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multiple paths in input_dir #1000

Merged
merged 3 commits into from
Feb 15, 2021
Merged

Support multiple paths in input_dir #1000

merged 3 commits into from
Feb 15, 2021

Conversation

jvegreg
Copy link
Contributor

@jvegreg jvegreg commented Feb 11, 2021

At BSC, we have different paths for variables generated directly from the model and those computed afterwards. In order to be able to read both, we need to pass multiple input_dirs because we need some replacements in the first part of the path.

An example:

EC-EARTH:
  input_dir:
    default:
      - '{expid}/cmorfiles/{activity}/{institute}/{dataset}/{exp}/{ensemble}/{mip}/{short_name}/{grid}/{latestversion}'
      - '{expid}/diags/{activity}/{institute}/{dataset}/{exp}/{ensemble}/{mip}/{short_name}/{grid}/{latestversion}'

Checklist

If you make backwards incompatible changes to the recipe format:

  • Update ESMValTool and link the pull request(s) in the description

@jvegreg jvegreg added the enhancement New feature or request label Feb 11, 2021
Copy link
Contributor

@sloosvel sloosvel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine.

@jvegreg jvegreg merged commit 2204338 into master Feb 15, 2021
@jvegreg jvegreg deleted the support_multiple_paths branch February 15, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants