Ticket #6338 (closed: fixed)
Statistics of a peaks workspace
Reported by: | Peter Peterson | Owned by: | Vickie Lynch |
---|---|---|---|
Priority: | major | Milestone: | Release 3.4 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Ricardo Ferraz Leal |
Description
As told to me at a Dec 5, 2012 meeting with the Mandi instrument scientist, Leighton Coates, implement a new algorithm to calculate characteristics of a peaks workspace according to the attached document.
Attachments
Change History
comment:4 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:5 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move to assigned at the introduction of the triage step
comment:6 Changed 6 years ago by Peter Peterson
Here is a link to the R pim stat formula
http://www.iucr.org/__data/iucr/cifdic_html/2/mmcif_pdbx.dic/Ireflns.pdbx_Rpim_I_all.html
Data completeness is the expressed as the percentage of all possible reflections for the given unit cell and space group That have been measured.
comment:8 Changed 6 years ago by Vickie Lynch
- Status changed from assigned to inprogress
Refs #6338 fixes for script to test statistics
Changeset: 87da341aa2ffdcfc8594032b905d2be17ab385d9
comment:9 Changed 6 years ago by Vickie Lynch
Refs #6338 fix isnan for windows
Changeset: 71a37a74b02652db568c018c97f9298512591f63
comment:10 Changed 6 years ago by Vickie Lynch
Refs #6338 statistics of peak workspace added
Changeset: e7b1f924fefd529768874a9d30b741f6998ce1d2
comment:11 Changed 6 years ago by Peter Peterson
I'm messing with some of this code trying to get the usage tests working again for SortHKL.
comment:12 Changed 6 years ago by Pete Peterson
Re #6338. Fixing usage test.
Changeset: 74938a35274e74f804e39c6e7164c8570d032524
comment:13 Changed 6 years ago by Vickie Lynch
Refs #6338 statistics in table workspace
Changeset: 8b980b71b0ac20da85e681837b7e2c593284d7db
comment:14 Changed 6 years ago by Vickie Lynch
Refs #6338 PredictPeaks needs all goniometers for completeness
Changeset: 1d85aebd6d60f5365fb5d8ad06b2e5e4fedb1465
comment:15 Changed 6 years ago by Vickie Lynch
Refs #6338 fix docs test
Changeset: 67c0e4a897e92d62722925e6fb010c5e290c0d97
comment:16 Changed 6 years ago by Vickie Lynch
Refs #6338 fix doc test
Changeset: 842ca591b50a0904c6419543110bf092bd9bbd60
comment:17 Changed 6 years ago by Vickie Lynch
Re #6338 running clang-format
Changeset: 132095fb21a360659f5fadc0aaebc6d26e776fee
comment:18 Changed 6 years ago by Vickie Lynch
Refs #6338 equivalent from all banks
Changeset: db9611e47381410f7782d19b215b2bbf49f096ba
comment:19 Changed 6 years ago by Vickie Lynch
Refs #6338 statisics for all runs separately
Changeset: be2dbecd349c42dea6bee418909a6de2a3271717
comment:20 Changed 6 years ago by Vickie Lynch
Refs #6338 new algorithm StatisticsOfPeaksWorkspace
Changeset: 84fda71152b92c7392e3b50b8bc27e312919c9ec
comment:21 Changed 6 years ago by Vickie Lynch
Refs #6338 options for resolution shells and banks plus tests
Changeset: 516d3e0bb573964ee3982bb6515a74a2039f7078
comment:22 Changed 6 years ago by Vickie Lynch
Refs #6338 fix doxygen and try parallel PredictPeaks
Changeset: 6e527d62b87265b8b84237bb110d7c169a69be22
comment:23 Changed 6 years ago by Vickie Lynch
Refs #6338 do not calculate completeness for each bank
Changeset: 97f8f8b82018d64bba109c85c7d8ceb8262728c0
comment:24 Changed 6 years ago by Vickie Lynch
- Status changed from inprogress to verify
- Tester set to ferrazlealrm@ornl.gov
- Resolution set to fixed
- Milestone changed from Backlog to Release 3.4
Would like Ricardo to test with protein crystallography data. Here is the script I used for testing. See attachments for input and formulas.
Load(Filename='Sapphire_Rhombohedral_R.peaks',OutputWorkspace='Sapphire_Rhombohedral_R') LoadIsawUB(InputWorkspace='Sapphire_Rhombohedral_R',Filename='Sapphire_Rhombohedral_R_copy.mat') StatisticsOfPeaksWorkspace(InputWorkspace='Sapphire_Rhombohedral_R', PointGroup='-3m1 (Trigonal - Rhombohedral)', OutputWorkspace='sorted', StatisticsTable='stat') StatisticsOfPeaksWorkspace(InputWorkspace='Sapphire_Rhombohedral_R', PointGroup='-3m1 (Trigonal - Rhombohedral)', OutputWorkspace='sortedRun', Sortby="RunNumber",StatisticsTable='statRun') StatisticsOfPeaksWorkspace(InputWorkspace='Sapphire_Rhombohedral_R', PointGroup='-3m1 (Trigonal - Rhombohedral)', OutputWorkspace='sortedBank',SortBy='Bank',StatisticsTable="statBank") StatisticsOfPeaksWorkspace(InputWorkspace='Sapphire_Rhombohedral_R', PointGroup='-3m1 (Trigonal - Rhombohedral)', OutputWorkspace='sortedOverall',SortBy='Overall',StatisticsTable="statOverall")
comment:25 Changed 6 years ago by Vickie Lynch
Refs #6338 try to fix cppcheck line 299
Changeset: 58e6565b767af0d5e69aac40a316b8bbd1427d1d
comment:26 Changed 6 years ago by Vickie Lynch
Refs #6338 suppress cppcheck syntaxError
Changeset: ca2d9bb2543efc595d914ca155bf024ffccc4533
comment:27 Changed 6 years ago by Vickie Lynch
Refs #6338 fix doxygen comment Changeset: 3507ca96a306a4a96ccfbd5016c4c182b79354e6
comment:28 Changed 6 years ago by Wenduo Zhou
- Status changed from verify to verifying
- Tester changed from ferrazlealrm@ornl.gov to Wenduo Zhou
comment:29 Changed 6 years ago by Vickie Lynch
- Status changed from verifying to reopened
- Resolution fixed deleted
TOPAZ instrument scientists tried this and said R's are too large and Completeness is too small.
comment:30 Changed 6 years ago by Vickie Lynch
- Status changed from reopened to inprogress
Refs #6338 fix multiplicity and sum all
Changeset: aaeda9a16d314c05c62fecaf21c04abd43c447fa
comment:31 Changed 6 years ago by Vickie Lynch
Refs #6338 SaveHKL OutputWorkspace with corrections
Changeset: 8eceaa575f55a59181ddc2555ba2767e4a0496cb
comment:32 Changed 6 years ago by Vickie Lynch
Refs #6338 fix doc test
Changeset: 3eb3724269edc0ce3d67902139eba31897ca1900
comment:33 Changed 6 years ago by Vickie Lynch
Refs #6338 change completeness
Changeset: ca461e320dd38987eb1b0d220fdabd2228e4d20a
comment:34 Changed 6 years ago by Vickie Lynch
Refs #6338 changes for merge
Changeset: ffa58d73b76c9aa93f97dadd1994d047e1b41b09
comment:35 Changed 6 years ago by Vickie Lynch
Refs #6338 fix doc tests
Changeset: cdbed07aa8d187df1a93077e8d9f1b2729840b85
comment:36 Changed 6 years ago by anonymous
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #450.
comment:37 Changed 6 years ago by Ricardo Ferraz Leal
- Status changed from verify to verifying
- Tester changed from Wenduo Zhou to Ricardo Ferraz Leal
comment:38 Changed 6 years ago by Vickie Lynch
Refs #6338 change precision for windows
Changeset: dbb8727241b207d9644e11d5b072a2a77562630d
comment:39 Changed 6 years ago by anonymous
retest this please
comment:40 Changed 6 years ago by anonymous
retest this please
comment:41 Changed 6 years ago by anonymous
reretest this please
comment:42 Changed 6 years ago by anonymous
Jenkins, retest this please.
comment:43 Changed 6 years ago by anonymous
Jenkins, retest this please
comment:44 Changed 6 years ago by anonymous
Jenkins, retest this please
comment:45 Changed 6 years ago by Vickie Lynch
- Status changed from verifying to closed
Merge branch 'feature/6338_peak_statistics' of github.com:mantidproject/mantid into feature/6338_peak_statistics
Full changeset: afdf1dbcfda258f3a1bd31b38d21de31f457cc77
comment:46 Changed 6 years ago by Ricardo Ferraz Leal
Merge pull request #450 from mantidproject/feature/6338_peak_statistics
Tested locally with Mandi data and I have sensible results.
Full changeset: 263d8cb509681b27f8033ed9339a0dc56d7743ac
comment:47 Changed 5 years ago by Nick Draper
Somehow these slipped through without a resolution. Set to Fixed.
comment:48 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7184