Ticket #7923 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:

  1. In the code, please use whitespace between math symbols and variables. Make it look like the code around it.
  2. In the ui.tex file, all references to buttons should be italicized, not quoted, to be consistent.
  3. In the ui.tex file, the paragraph before section 3.2 should use the \nexus{} tex code instead of plain text.
  4. In the same paragraph above, the Save to folder expression should be italicized since it refers to a UI element.
  5. 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.
  6. 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:10 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying

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:14 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying

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

Note: See TracTickets for help on using tickets.