-
Notifications
You must be signed in to change notification settings - Fork 102
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
Fixed bug in the Start of Rains Dialog #7492
Fixed bug in the Start of Rains Dialog #7492
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@N-thony this looks to be fixed. So I am approving. I hope that @ChrisMarsh82 can check and merge.
I did notice that the dialogue ignores a filter, if one is applied. I checked back and this is also the case in Version 0.7.4, so perhaps to be examined later.
@N-thony I tested Climatic > Prepare > Spells dialogue and works fine. |
@N-thony I tested and it is now working. Thank you |
@shadrackkibet could you review this? Its R code which would be better for you to approve |
@N-thony on second thought the above changes make more sense. The previous method wasn't working when this bit is called in the `summary_tables` function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rdstern if you test and re-approve then we can merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a big relief. I have been using it on data from Zambia and it seems to work fine now.
Fixes #7488
@lilyclements thanks for your support, @rdstern I had a quick look of it then requested help from Lily, this seems to be fixed now, please take a look.
@derekagorhom @VimbayiChimonyo could you also test?
@dannyparsons could you review this? Thanks.