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

Corrections and an oddity in the climatic data entry dialogue #7279

Closed
rdstern opened this issue Mar 8, 2022 · 6 comments
Closed

Corrections and an oddity in the climatic data entry dialogue #7279

rdstern opened this issue Mar 8, 2022 · 6 comments
Assignees
Labels
Milestone

Comments

@rdstern
Copy link
Collaborator

rdstern commented Mar 8, 2022

I have been using the Climatic > Tidy and Examine > Daily Data Entry dialogue as an editing facility to correct daily data for 2 stations in Tajikistan. This is following the quality control, that showed a number of oddities. For example there were 97 occasions with tmax <= tmin. I now need to correct the data. I am unable to get access to the paper records, and need to see the odd days in context.
I am finding the data entry tool to be excellent for this in a variety of ways.

image

There are a couple of small improvements I suggest, and then an oddity that is shown here - a bug, perhaps to be found.
a) Please change the menu name to Daily Data Editing/Entry .
b) And make the same change in the title of the dialogue.
c) When you return to the data entry after coming out of data entry it always asks if you really want to, because you will lose what was entered earlier. Here is an example of the message:
image

Now, once you have entered, or corrected data, the number of corrections is shown in red at the bottom. But the message appears whether, or not, you have made any corrections. In the example above, I went into the sub-dialogue, then came straight out, and am now returning. The message should not be given unless there are values to correct.

I hope these are quick and easy to do, because they include an edit to frmmain.

d) There is a bug that has just appeared. It is that yellow box in the figure above. This has appeared and is very annoying. It is very persistent. I have reset the main dialogue, and also the sub-dialogue. But it has just appeared (I now realise it is in the master I should say - I was using Version 7.3 earlier??? and didn't get this!) I'll start again - with the master, and see if it is a consistent problem. Then - if so, I'll check with other versions.

I am suggesting @N-thony could fix at least a) to c) easily. I really don't understand d) and will check further.

@N-thony
Copy link
Collaborator

N-thony commented Apr 16, 2022

@rdstern I'm not able to reproduce item d), please check again otherwise we can close this?

@rdstern
Copy link
Collaborator Author

rdstern commented May 26, 2022

@N-thony In the 2 recent workshops this dialogue seems useful. I still need to check for the oddity above and suggest we may find more quirks as we use it more. So while a developer makes small improvements I hope that @Wycklife can make serious use of the dialogue, and I think more oddities will appear. It is quite sensible to use it for the entry of some of the additional data from the Kenya stations and also to make corrections. Here are small tasks so far.

a) The default is by month. Go into that and (if you change a day, or more) and add a comment - or two - then you can update, i.e. press Ok. But I sometimes don't want to change, just add a comment. I would like to update the comments even if just a comment is added. Currently this is not possible.
b) When I click on Range and choose, e.g. 3 months it does not keep that setting. It resets to Month when I re-open the dialogue.
c) When using range I was not able to add a comment the add comment was disabled.

I will be surprised if we don't find more as we use it.

@N-thony
Copy link
Collaborator

N-thony commented May 30, 2022

@N-thony In the 2 recent workshops this dialogue seems useful. I still need to check for the oddity above and suggest we may find more quirks as we use it more. So while a developer makes small improvements I hope that @Wycklife can make serious use of the dialogue, and I think more oddities will appear. It is quite sensible to use it for the entry of some of the additional data from the Kenya stations and also to make corrections. Here are small tasks so far.

a) The default is by month. Go into that and (if you change a day, or more) and add a comment - or two - then you can update, i.e. press Ok. But I sometimes don't want to change, just add a comment. I would like to update the comments even if just a comment is added. Currently this is not possible. b) When I click on Range and choose, e.g. 3 months it does not keep that setting. It resets to Month when I re-open the dialogue. c) When using range I was not able to add a comment the add comment was disabled.

I will be surprised if we don't find more as we use it.

@EstherNjeriLiberatta please implements items a) and b). c) isn't a problem, you just need to define key before adding a comment in the data, perhaps add a tooltip to Add comment in the Comment subdialogue.

@EstherNjeriLiberatta EstherNjeriLiberatta self-assigned this May 31, 2022
@N-thony
Copy link
Collaborator

N-thony commented Jun 2, 2022

@rdstern I think the issue in item b) is due to the translation, so I suggest that @EstherNjeriLiberatta after fixing item a), to open another PR that will only contain the change in the database.

@N-thony
Copy link
Collaborator

N-thony commented Jun 17, 2022

@N-thony In the 2 recent workshops this dialogue seems useful. I still need to check for the oddity above and suggest we may find more quirks as we use it more. So while a developer makes small improvements I hope that @Wycklife can make serious use of the dialogue, and I think more oddities will appear. It is quite sensible to use it for the entry of some of the additional data from the Kenya stations and also to make corrections. Here are small tasks so far.

a) The default is by month. Go into that and (if you change a day, or more) and add a comment - or two - then you can update, i.e. press Ok. But I sometimes don't want to change, just add a comment. I would like to update the comments even if just a comment is added. Currently this is not possible. b) When I click on Range and choose, e.g. 3 months it does not keep that setting. It resets to Month when I re-open the dialogue. c) When using range I was not able to add a comment the add comment was disabled.

I will be surprised if we don't find more as we use it.

@rdstern for the item a), I think if I do remember well, in the implementation of this dialogue the idea was when entering the data, we wanted to facilitate user to add comment while entering the data rather than entering data and then going back and use the normal add comment dialogue, I think if you want to add a comment without entering the data you can just use the normal add comment dialogue. Just trying to raise this before making the change.

@N-thony
Copy link
Collaborator

N-thony commented Jul 11, 2022

@lilyclements @EstherNjeriLiberatta could we have a quick discussion today?

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

No branches or pull requests

3 participants