Ticket #3666 (closed: fixed)
Provide an algorithm to choose a conventional cell given a reduced cell. (Scalar)
Reported by: | Dennis Mikkelson | Owned by: | Dennis Mikkelson |
---|---|---|---|
Priority: | major | Milestone: | Release 2.1 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Janik Zikovsky |
Description
After a proper Niggli reduced cell has been found, it is necessary in many cases to allow the user to choose a corresponding conventional cell.
Change History
comment:2 Changed 9 years ago by Dennis Mikkelson
Added support class for finding conventional cell.
refs #3666
Changeset: 13b13701c54f1b322c0bb203489b9848e11dad1b
comment:3 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:4 Changed 9 years ago by Dennis Mikkelson
Added ScalarUtils class, to support choosing conventional cell
Added new class (ScalarUtils), tests and some updates to supporting classes. This puts in place the main classes needed for algorithms to choose conventional cells given a Niggli reduced cell. refs #3666
Changeset: f0a2ee6440fbe7b728a5415af6565876b97f1bbc
comment:5 Changed 9 years ago by Dennis Mikkelson
Replace fmin with std::min for windows.
fmin not found in windows builds. refs #3666
Changeset: 7ebfa256f35b37f86c0777e9016b96f0cc9bf4e1
comment:6 Changed 9 years ago by Dennis Mikkelson
Replace Kernel::DblMatrix with DblMatrix for Windows build
refs #3666
Changeset: 8d54b921457a2ba4d9d69d88ca36221068a83f90
comment:7 Changed 9 years ago by Dennis Mikkelson
Remove unused exception variable (fix windows warning)
refs #3666
Changeset: e2a59e009250a564a451edbd24563f003e9280a5
comment:8 Changed 9 years ago by Dennis Mikkelson
Add initial version of ShowPossibleCells
ShowPossibleCells displays a list of possible conventional cells corresponding to a Niggli reduced cell UB in a peaks workspace. refs #3666
Changeset: a4dc2d3e8abcfc4aa47cce0b68bc90661f540baa
comment:9 Changed 9 years ago by Dennis Mikkelson
Add GetLatticeParamterString() convenience method
refs #3666
Changeset: f3168d2a81bfcf885e1f11a0644b56bb558041e5
comment:10 Changed 9 years ago by Dennis Mikkelson
Removed redundant clone() of BoundedValuator
refs #3666
Changeset: cfcc79433895c07232292fd090df7136eb296ceb
comment:11 Changed 9 years ago by Dennis Mikkelson
comment:12 Changed 9 years ago by Dennis Mikkelson
Add GetLatticeParamterString() convenience method
refs #3666
Changeset: f3168d2a81bfcf885e1f11a0644b56bb558041e5
comment:13 Changed 9 years ago by Dennis Mikkelson
Removed redundant clone() of BoundedValuator
refs #3666
Changeset: cfcc79433895c07232292fd090df7136eb296ceb
comment:14 Changed 9 years ago by Dennis Mikkelson
comment:15 Changed 8 years ago by Dennis Mikkelson
- Status changed from new to accepted
This will be done by a combination of three algorithms:
ShowPossibleCells--Show possible conventional cell corresponding to a Niggli reduced cell.
SelectCellOfType--Change the UB corresponding to a Niggli reduced cell to a conventional cell with a specified lattice type and centering.
SelectCellWithForm--Change the UB corresponding to a Niggli reduced cell to the conventional cell with a particular form number, as listed by ShowPossibleCells.
NOTE: All of these methods only apply to the UB stored in the PeaksWorkspace, provided that UB corresponds to a Niggli reduced cell. A Niggli cell is produced by FindUBUsingFFT or FindUBUsingMinMaxD. If the cell is NOT a Niggli cell, the list of possible conventional cells will be non-sense.
comment:16 Changed 8 years ago by Dennis Mikkelson
- Status changed from accepted to verify
- Resolution set to fixed
comment:17 Changed 8 years ago by Janik Zikovsky
- Status changed from verify to verifying
- Tester set to Janik Zikovsky
comment:18 Changed 8 years ago by Janik Zikovsky
- Status changed from verifying to closed
OK I've used all 3 of these algorithms with real TOPAZ data and it all works very nicely; as long as you know that you need to use FindUBUsingFFT, then ShowPossibleCells, then SelectCellOfType.
I've made ticket #5227 to roll all of these methods into one friendly GUI.
comment:19 Changed 8 years ago by Janik Zikovsky
BTW: Is there a way to test if the input unit cell is a Niggli reduced cell and give a warning if it isn't?
comment:20 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4513
Bulk move of tickets to iteration 31 at the iteration 30 code freeze