ljramones / trips

A stellar cartography system
Apache License 2.0
15 stars 2 forks source link

Change the load dialogue to make the load messages more visible #342

Closed ljramones closed 2 years ago

ljramones commented 2 years ago

as it says

ljramones commented 2 years ago

THese are Chuck's comments

Selected 270 LY csv file for loading/adding dataset. Process was under way. It indicated that 20000 of 20000 stars had loaded. I clicked on it. The table disappeared. I looked for the dataset. It was not there. I figured the load had failed and tried to add it again. TRIPS told me it was still loading. However, there was no box or other processing advice/warning on the display to indicate this at all. It finally loaded and plotted when I went away. I think if a process is still underway, there needs to be something on the display which indicates that. Maybe blinking if it’s actually pausing/impeding other activity.

ljramones commented 2 years ago

load dialogue is now more responsive. I update the feedback more by dropping the report rate from 20,000 stars per update to 2,000 stars per update I put the loading message in a larger bold red font. I think its fixed and if needs be, we can reopen

cegannon1 commented 2 years ago

The only way it was misleading is that it had loaded

"20000 of 20000 stars"

...which made no sense.

I would normally expect the second value to be actual total records to load, as in:

"20000 of 140000"

Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: L Mitchell @.> Sent: Saturday, January 15, 2022 5:45:39 PM To: ljramones/trips @.> Cc: Subscribed @.***> Subject: Re: [ljramones/trips] Change the load dialogue to make the load messages more visible (Issue #342)

load dialogue is now more responsive. I update the feedback more by dropping the report rate from 20,000 stars per update to 2,000 stars per update I put the loading message in a larger bold red font. I think its fixed and if needs be, we can reopen

— Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/342#issuecomment-1013766101, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNRWMMESQBPCZBIRHETUWH2JHANCNFSM5MAIJRYQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.***>

ljramones commented 2 years ago

There is no way that I can apriori know how many records are going to be loaded. Originally, I would have the first value be the number of records per update and the second number be the cumulative total. So 2000 of 2000 then 2000 of 4000. I decided to just give a cumulative total. I can't show how far along it will be.

ljramones commented 2 years ago

It actually will show you 20000 of 140000 if you wait long enough. Early on, it loaded very quickly, but since we increased the complexity of each line and number of data items, its really slowed down. On a less powerful computer, the time between updates is going to look like it's hung and not doing anything. I changed the load factor form 20,000 to 2,000 to give a more responsive update message.

cegannon1 commented 2 years ago

If I understand what you’re saying (and I might not be), then how does it help a user understand “how far along” the process is?

From: L Mitchell @.> Sent: Saturday, January 15, 2022 6:03 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Comment @.> Subject: Re: [ljramones/trips] Change the load dialogue to make the load messages more visible (Issue #342)

There is no way that I can apriori know how many records are going to be loaded. Originally, I would have the first value be the number of records per update and the second number be the cumulative total. So 2000 of 2000 then 2000 of 4000. I decided to just give a cumulative total. I can't show how far along it will be.

— Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/342#issuecomment-1013767980, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNRHKYIRDSJ5JLR65FLUWH4JFANCNFSM5MAIJRYQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you commented.Message ID: @.**@.>>