PSU-CSAR / vb-bagis-p

VB .NET source code for ArcMap BAGIS Parameters add-in
1 stars 1 forks source link

Verification process interrupted when users (repeatedly) click-on the methods grid #37

Open jdduh opened 7 years ago

jdduh commented 7 years ago

Users who have a slow computer are more likely to encounter this problem. They tried to see the progress of the verification but ended up getting this error message.

image

lbross commented 7 years ago

Do you know what they were doing with the grid? Sorting? clicking the scroll bars? I noticed that the 'use' checkboxes are enabled when the calculation is running which isn't good. I will fix that.

jdduh commented 7 years ago

They just wanted to move the scroll bar to view the verification progress.

lbross commented 7 years ago

The good news is, I was able to replicate the behavior. The bad news is, I can't think of a good way to address it. For some reason, when you move the scroll bars around, it sends a cancel message to the Geoprocessor. The Geoprocessor calls a model to create the parameter table. Here is the error message:

GP Msg: Cancelled function GP Msg: (Table to Table) aborted by User. GP Msg: Failed at Mon Oct 24 15:43:19 2016 (Elapsed Time: 0.98 seconds)

I can disable the grid while the verification process is running which will prevent the failure, but it also means that the scroll bars won't work :-1:

jdduh commented 7 years ago

What will happen if the user tries to sort the cells? Could it be that the ValueChanged event of the scrollbar attempts to refresh/reload the gridview? When BAGIS-P calculates the parameters, the scrollbar seems to perform normally so that the users can view the entire method list without cancelling out the calculation.

lbross commented 7 years ago

The sort function causes the error as well. I found that the GP tool that was being used to create the parameter table was called using a different method than we use when calculating the actual parameters. I switched it over to use the same method and I'm no longer getting the error. However, the scrollbars don't seem to work either. I'm not sure why this is different than we see with the parameter calculation.

The original method had a TrackCancel argument that was set to cancel on click. Perhaps this opens the UI up to responding to clicks?

lbross commented 7 years ago

@jdduh I have tried everything I can think of to get the scroll bars to work with the verify button, but they don't work unless the TrackCancel argument accepts clicks and then we get the exception when trying to create the parameters table. OK to close this with scrollbars not usable when verifying?

jdduh commented 7 years ago

Is there a way to make it clear to the users that the scrollbar is disabled? For example. grey it out or hide it. How about creating a modal dialog window asking the user to wait? I would like to see the scrollbar remains active during parameter calculations.

lbross commented 7 years ago

Would you prefer hiding the scrollbar or creating a modal dialog window asking the user to wait during verification? I'm not going to do anything with the scrollbar during the parameter calculations because that is currently working as desired.

jdduh commented 7 years ago

The verification usually doesn't take very long. How about a modal dialog window, saying "Verifying methods. Please wait!"?

lbross commented 7 years ago

I added a label that overlays the method box with the warning message. Easier than another window and visually pleasing. This will be in the next BAGIS-P releasel