Ticket #6493 (closed: fixed)
Consistent Glyphs
| Reported by: | Owen Arnold | Owned by: | Owen Arnold |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.4 |
| Component: | VATES | Keywords: | |
| Cc: | reuterma@… | Blocked By: | |
| Blocking: | #6514 | Tester: | Michael Reuter |
Description
In the PeaksViewer addition for the SliceViewer http://www.mantidproject.org/PeaksViewer, only PeaksWorkspaces that are integrated (and only some of those at the moment)are shown with a spherical shell. Otherwise the peak position is marked with a cross position. This behaviour is not consistent with the PV peaks reader and source plugins. In the current PV plugins, even PeaksWorkspaces that are not integrated are shown with a spherical shell. Clearly plotting a shell is wrong in this case, because there's no correct radius with which to set the shell. Furthermore, it now gives the impression that a PeaksWorkspace has been integrated even when it hasn't.
This ticket is to make the PV plugins consistent with the PeaksViewer.
Change History
comment:2 Changed 8 years ago by Owen Arnold
- Cc reuterma@… added
- Status changed from new to accepted
comment:3 Changed 8 years ago by Owen Arnold
refs #6493. Use glyphs to differentiate.
Changeset: e55f5c90fec6ce13e7d35bfb0fc683b2977d81db
comment:4 Changed 8 years ago by Owen Arnold
- Status changed from accepted to verify
- Resolution set to fixed
Tester:
- Load an unintegrated peaks workspace into the VSI (glyphs should be cross-hairs).
- Run IntegratePeaksMD on the peaks workspace and load into the VSI (glyphs should be spheres)
Do the same thing to check the plugins directly in ParaView.
comment:6 Changed 8 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:7 Changed 8 years ago by Michael Reuter
- Status changed from verifying to closed
This looks to be working correctly.
comment:10 Changed 8 years ago by Owen Arnold
refs #6493. Use glyphs to differentiate.
Changeset: e55f5c90fec6ce13e7d35bfb0fc683b2977d81db
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7339
I've approved with Dennis, Michael and Nick that I'm going to squeeze this into this release (2.4)