-
Notifications
You must be signed in to change notification settings - Fork 5
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
ws vocabularies > 680 #359
Comments
@vmarchman @mcfrank I've tried fixing this many times and many ways without success so I propose a different approach. Instead of doing the calculation during form filling or at the end, we move it to an action for the researcher. We'll give then a checkbox to highlight those they want to process and we'll do the processing in the background and send them a message once complete. This still runs the risk of a researcher running the process twice at the same time, so I'll think about how to sort that at the same time if you're happy with the approach |
@Henry ***@***.***> How does this solution affect the graph at the
end?
…On Mon, Aug 8, 2022 at 2:29 AM Henry Mehta ***@***.***> wrote:
@vmarchman <https://github.com/vmarchman> @mcfrank
<https://github.com/mcfrank>
The way this program works is when the calculation starts, values are set
to 0 and then amended incrementally as records are checked. The problem is
sometimes, and I think this is people double clicking the finish button,
the calculations are performed twice and the second overlaps the first. So
the first sets values to zero, starts amending them, then the second sets
values to zero, first and second continue to amend, first finishes and
second finishes.
I've tried fixing this many times and many ways without success so I
propose a different approach. Instead of doing the calculation during form
filling or at the end, we move it to an action for the researcher. We'll
give then a checkbox to highlight those they want to process and we'll do
the processing in the background and send them a message once complete.
This still runs the risk of a researcher running the process twice at the
same time, so I'll think about how to sort that at the same time if you're
happy with the approach
—
Reply to this email directly, view it on GitHub
<#359 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB2TUTB67A25RKY2IU4FOITVYDHN3ANCNFSM5DXPVWZA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@vmarchman blast, I had forgotten about that. Will think again |
@vmarchman @mcfrank I've one more idea which I got while typing the above sentence which I will try once I've sent this. In the meantime, I'm running an update to correct all the existing values and ensure they're ok |
seems like when we download data from WS English (American) we get some kids with Total_Produced > 680. maybe @vmarchman can give a reprex or example of subjects that have this issue?
The text was updated successfully, but these errors were encountered: