Ticket #7923 (closed: fixed)
DGSreduction cross section
Reported by: | Andrei Savici | Owned by: | Andrei Savici |
---|---|---|---|
Priority: | trivial | Milestone: | Release 3.0 |
Component: | GUI | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Michael Reuter |
Description
Add comment about cross section in DGS absolute reduction
Change History
comment:1 Changed 7 years ago by Andrei Savici
- Status changed from new to inprogress
- Owner set to Andrei Savici
comment:2 Changed 7 years ago by Andrei Savici
Use the vanadium cross section already in Mantid. Refs #7923
Changeset: 25a1150cce7a62fbc21fac02ae578afe8b072641
comment:3 Changed 7 years ago by Andrei Savici
Updated document for DGSReduction. Refs #7923
Added comments about absolute normalization, multifile loading, live stream, and cluster submission
Changeset: 00e7bd43a79377a3b39718c9a657e7d3746081ad
comment:4 Changed 7 years ago by Andrei Savici
- Status changed from inprogress to verify
- Resolution set to fixed
comment:5 Changed 7 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:6 Changed 7 years ago by Michael Reuter
- Status changed from verifying to reopened
- Resolution fixed deleted
I have a few minor complaints that I'd like to see addressed:
- In the code, please use whitespace between math symbols and variables. Make it look like the code around it.
- In the ui.tex file, all references to buttons should be italicized, not quoted, to be consistent.
- In the ui.tex file, the paragraph before section 3.2 should use the \nexus{} tex code instead of plain text.
- In the same paragraph above, the Save to folder expression should be italicized since it refers to a UI element.
- In the ui.tex file, in the first paragraph of section 3.5 (remote jobs), DGSReduction should be italicized and split into two words to be consistent with rest of text.
- In the same paragraph above selectthe should be select the.
As a general observation on the documentation, make sure that UI element references are italicized.
comment:7 Changed 7 years ago by Andrei Savici
- Status changed from reopened to inprogress
Refs #7923. Minor edit
Changeset: 693af9202ff7c8376971d18f6011d90e33d7e871
comment:8 Changed 7 years ago by Andrei Savici
Refs #7923. Minor document edits
Changeset: 0b05a3081338bbbff05cd0f496ff4be77f93c066
comment:9 Changed 7 years ago by Andrei Savici
- Status changed from inprogress to verify
- Resolution set to fixed
comment:11 Changed 7 years ago by Michael Reuter
- Status changed from verifying to reopened
- Resolution fixed deleted
Missed one UI element: Send to cluster in sentence 3 in only paragraph of section 3.5. :-)
comment:12 Changed 7 years ago by Andrei Savici
- Status changed from reopened to inprogress
Refs #7923. Minor edit
Changeset: bd45745ac602977c10f0e90f94387f21a0baf649
comment:13 Changed 7 years ago by Andrei Savici
- Status changed from inprogress to verify
- Resolution set to fixed
comment:15 Changed 7 years ago by Michael Reuter
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/7923_DGS_comment'
Full changeset: 5d4cc7339bf0fe94a30a6285f86228faa43449bd
comment:16 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8768