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

Error messages from start of rains analysis using shifted climatic data #9185

Open
jkmusyoka opened this issue Oct 10, 2024 · 2 comments · May be fixed by #9175
Open

Error messages from start of rains analysis using shifted climatic data #9185

jkmusyoka opened this issue Oct 10, 2024 · 2 comments · May be fixed by #9175
Assignees

Comments

@jkmusyoka
Copy link
Contributor

jkmusyoka commented Oct 10, 2024

@N-thony for you to check quickly. Seems something small to fix I hope

There are 2 error messages that have popped up during the start of rains analysis with shifted data here in Eswatini.

The first message below appeared when opening the start of rains dialog. Clicking OK still opened the dialog

image

Submitting a filled-in start of rains dialog then gave the following second error message. Again closing the message box still gave the output in a new dataframe as expected.

image

The dialog seems to work despite the errors thats why I say it may be a small job. And so we do need to find out why the errors and fix the issues

@rdstern
Copy link
Collaborator

rdstern commented Oct 10, 2024

@Vitalis95 I know it is a holiday today in Kenya, so I'll do the first check.

This dialog has changed recently to add the status (logical) variable and now is changing again. I'll check on the new pull request and report. The first error has happened (and been fixed) on quite a number of dialogs. The second is new. @jkmusyoka usually when we get the annual summaries we first get seasonal totals and so on. Then add the start of the rains to the same dataframe. We should be able to start the yearly data frame with the start dialog. Is that what you are doing?

@MeSophie
Copy link
Contributor

@rdstern @jkmusyoka The version of Start of Rain dialog that we have on PR #9175 doesn't produce the first error and I've detected where the second came from so I'll see how to fix it with @Vitalis95.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants