-
Notifications
You must be signed in to change notification settings - Fork 129
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
Flatten ancestor lists for diag_spei.R and diag_spi.R. #1745
Conversation
Not completely tested due to Mistral issues. |
You still can run the recipe in interactive mode (better on mistralpp). |
Yes, I did that yesterday, but it took forever (and I forgot something). I can restart it also again in there. |
Searching for
Do you see anything that can be quickly fixed in a similar way? Sorry if I can't help, but I don't speak R. 😞 |
Tested successfully! |
Merging this now. |
ok ,thanks! |
Before you start, please read our contribution guidelines.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.
Tasks
yamllint
to check that your YAML files do not contain mistakesNew recipe/diagnostic
doc/sphinx/source/recipes
folder and add a new entry to index.rstModified recipe/diagnostic
doc/sphinx/source/recipes
folderNew data reformatting script
Modified data reformatting script
If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.
Closes #1717