Ticket #9032 (closed: fixed)
Get rid of warning message from 8970
Reported by: | Gesner Passos | Owned by: | Gesner Passos |
---|---|---|---|
Priority: | major | Milestone: | Release 3.1.1 |
Component: | SANS | Keywords: | PatchCandidate |
Cc: | Blocked By: | ||
Blocking: | Tester: | Peter Parker |
Description
Deal with warning on SANS due to ticket #8970
Workspace already exists in a WorkspaceGroup Workspace already exists in a WorkspaceGroup
Do a reduction twice and you will get this warning because it is trying to add to a WorkspaceGroup the same name.
Change History
comment:2 Changed 7 years ago by Gesner Passos
- Keywords PatchCandidate added
Because 8970 is PatchRelease, this completes the solution, and should also go published together.
comment:3 Changed 7 years ago by Gesner Passos
re #8970 Check ws belongs to group before inserting it
This get rids of warning message in SANS reduction due to inserting workspace to a group which already had that workspace.
Changeset: 74f11af570fc0755ce639dc7377fd694bc92df6d
comment:4 Changed 7 years ago by Gesner Passos
The commit message was with the wrong number 8970, but if you use git macro, you won't find any problem testing this ticket.
comment:5 Changed 7 years ago by Gesner Passos
Tester:
- open SANS Interface
- select SANS2D instrument
- load MaskSANS2DReductionGUI.txt user file
- change to batch mode
- load sans2d_reduction_gui_batch.csv
- Run 1D reduce
- Check that in the workspace widget, you will be able to see under sample_can_reduction the temporary workspaces
- Change to single mode
- Run 1D reduce
- In the previous version, you would see the warning message, after this ticket you should see it no more.
comment:6 Changed 7 years ago by Gesner Passos
- Status changed from assigned to verify
- Resolution set to fixed
comment:7 Changed 7 years ago by Peter Parker
- Status changed from verify to verifying
- Tester set to Peter Parker
comment:8 Changed 7 years ago by Nick Draper
- Milestone changed from Release 3.2 to Release 3.1.1
Moved to patch release 3.1.1
comment:9 Changed 7 years ago by Peter Parker
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/9032_warning_msg'
Full changeset: 680af0df21b18b52038f40b6a9b2f4ee936c7059
comment:10 Changed 7 years ago by Russell Taylor
This has now been merged to the patch release branch (https://github.com/mantidproject/mantid/commit/537dd039c47d38463d145e12deb8b6907f2fa394)
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9875