A couple of code doc changes to loadmcstas

[Anders Markvardsen]

Torben writes:

Derudover er der 2 slå fejl som kan rettes en gang når du/vi får tid: LoacMcStasTest.h linje 25 skal være: Test checks if number getNumberHistograms = 2x4096. (64x64= 4096 pixels in one detector)

Dvs. så er regnestykket på 8192 pixels mere sigende.

LoadMcStas.cpp linje 7 skal være LoadMcStas can be used for reading McStas 2.1 histogram and event data. Dvs. event var stavet forkert. (evnet –>event )

[Anders Markvardsen]

Torben writes:

#xyzw: Deleting redundant lines in LoadMcStas.cpp

It looks like line 333 – 334 can be deleted.

[Anders Markvardsen]

Doc changes to loadmcstas. re #8272

Changeset: 329b36f8aef125160f5e4a3de2eb4801bddebfc6

[Anders Markvardsen]

To test:

check that

1. LoacMcStasTest.h linje 25 now reads:

Test checks if number getNumberHistograms = 2x4096. (64x64= 4096 pixels in one detector)

2. LoadMcStas.cpp linje now reads: LoadMcStas can be used for reading McStas 2.1 histogram and event data.

[Karl Palmen]

The only change is the correction of the line

"LoadMcStas can be used for reading McStas 2.1 histogram and event data." in the wiki statement.

Line 25 of the unit test reads "Test checks if number getNumberHistograms = 327682x16384. (128x128= 16384 pixels in one detector)", which is different than what is required in part 1 of the test described above.

[Anders Markvardsen]

remaining code doc for this ticket. re #8272

Changeset: 0a058c9b235e252db85619c7f51a69884fa5113a

[Anders Markvardsen]

Karl thanks for spotting this. To test see description above

[Karl Palmen]

It needs a checkbuild done before testing can be done.

[Anders Markvardsen]

Checkbuild done. To test see description above

[Karl Palmen]

Merge remote-tracking branch 'origin/feature/8272_doc_change_loadmcstas'

Full changeset: ​26086e389cf9d12f2efa887cfce294f89a024626

[Stuart Campbell]

This ticket has been transferred to github issue ​9117