Skip to content

Fix input file error message#374

Merged
forsyth2 merged 1 commit intomainfrom
issue-125-input-error
Dec 6, 2022
Merged

Fix input file error message#374
forsyth2 merged 1 commit intomainfrom
issue-125-input-error

Conversation

@forsyth2
Copy link
Collaborator

@forsyth2 forsyth2 commented Dec 6, 2022

Fix input file error message. Resolves #125.

@forsyth2 forsyth2 added the priority: low Low priority task label Dec 6, 2022
@forsyth2 forsyth2 self-assigned this Dec 6, 2022
@forsyth2
Copy link
Collaborator Author

forsyth2 commented Dec 6, 2022

Tested by setting years = "1500:1502:2", (e.g., years that would not be in the data) for [climo] [[ atm_monthly_diurnal_8xdaily_180x360_aave ]] and [ts] [[ atm_monthly_180x360_aave ]].

@forsyth2
Copy link
Collaborator Author

forsyth2 commented Dec 6, 2022

The issue was that if ls {{ case }}.{{ input_files }}.????-*.nc does not match any files, then input.txt will be filled with files ending in *.nc rather than actual input files. Thus, we run grep on the *.

@forsyth2 forsyth2 merged commit 9a160e6 into main Dec 6, 2022
@forsyth2 forsyth2 deleted the issue-125-input-error branch December 6, 2022 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: low Low priority task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add error message when input files are not found

1 participant