Ticket #6032 (closed: worksforme)

Opened 8 years ago

Last modified 5 years ago

Progress Update for Deletion

Reported by: Owen Arnold Owned by: Karl Palmen
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description (last modified by Karl Palmen) (diff)

Loading a large workspace can take some time, and we correctly provide a progress update to our users when this happens.

Deleting a large workspace can also take some time, but we do not report progress here.

Change History

comment:1 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

Moved at the code freeze for release 2.4

comment:2 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:3 Changed 7 years ago by Karl Palmen

  • Status changed from new to assigned
  • Owner changed from Anyone to Karl Palmen

comment:4 Changed 7 years ago by Karl Palmen

A look at the code shows it would be quite easy to put the progress reporting into the destructor of workspace2D, but this would disrupt any progress reporting done by calling algorithm.

comment:5 Changed 7 years ago by Karl Palmen

The destructor of ManagedWorkspace2D also has a loop that could support progress reporting and it may be here that the progress reporting is most needed. Again it would disrupt progress reporting of any calling algorithm.

comment:6 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:8 Changed 7 years ago by Karl Palmen

  • Status changed from new to verify
  • Resolution set to worksforme
  • Description modified (diff)

To test. If you can create a workspace that takes time to delete, create and delete this workspace. If it takes consideranle time to delete, fail the test and pass details to this ticket.

comment:9 Changed 7 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:10 Changed 7 years ago by Nick Draper

  • Status changed from verifying to closed

This should really be a wontfix.

I think some of the point has been missed as I think the time spent is in releasing large quantities of memory rather than within code we have direct control over.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6878

Note: See TracTickets for help on using tickets.