Issue 772: ImportTransitionListColumnSelectDlg.CheckForErrors() needs to use a LongWaitDlg

issues
Status:closed
Assigned To:Guest
Type:Defect
Area:Skyline
Priority:1
Milestone:21.1
Opened:2021-03-02 17:27 by Brendan MacLean
Changed:2021-06-23 14:52 by Brian Pratt
Resolved:2021-06-23 14:52 by Brian Pratt
Resolution:Fixed
Closed:2021-06-23 14:52 by Brian Pratt
2021-03-02 17:27 Brendan MacLean
Title»ImportTransitionListColumnSelectDlg.CheckForErrors() needs to use a LongWaitDlg
Assigned To»Brian Pratt
Type»Defect
Area»Skyline
Priority»1
Milestone»21.1
I just tried for the first time to import a 450K line assay library, and while the new ImportTransitionListColumnSelectDlg popped right up and showed me useful information, when I clicked OK (or it could have been Check for Errors) Skyline hung for long enough for me to tire of waiting and click the Pause button in the debugger to see the that I was stuck in the CheckForErrors() function. Hopefully this actually does all of the necessary processing during that check, but it still needs to show a LongWaitDlg with progress the way the old code did before we had to show the column selection form.

2021-03-02 17:40 Brendan MacLean
Well, actually, it takes 8-10 seconds for the form to show up. So, we need a LongWaitDlg on that side of the operation as well.

2021-03-03 10:01 Brendan MacLean
And, unfortunately, check for errors is now separated from importing the assay library. So, you see all the errors and then you get a LongWaitDlg saying everything is being imported. At least you don't get the errors again, but in the old model the errors appeared at the end of the import progress.

It would be good to time how long it takes to process the import now versus the time it took for 20.2 (subtracting the time in the new form).

2021-06-23 14:52 Brian Pratt
resolve as Fixed
Statusopen»resolved
Fixed 3/29/2021 in commit dcea4e65 "Add a progress message for preprocessing step of import of large Assay Libraries and Transition Lists (#1488)"

2021-06-23 14:52 Brian Pratt
close
Statusresolved»closed
Assigned ToBrian Pratt»Guest