Ticket #10499 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Add RRFMuon algorithm

Reported by: Raquel Alvarez Banos Owned by: Raquel Alvarez Banos
Priority: major Milestone: Release 3.4
Component: Muon Keywords:
Cc: anders.markvardsen@… Blocked By: #10498
Blocking: Tester: Karl Palmen

Description (last modified by Raquel Alvarez Banos) (diff)

Add RRFMuon algorithm where the name of the .h and .cpp will be RRFMuon.h/.cpp.

The input is that of ticket #10498 plus the frequency of the oscillations.

The output should be the envelope of the decay.

Add unit test, .rst file with usage example.

Change History

comment:1 Changed 6 years ago by Anders Markvardsen

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Raquel Alvarez Banos

  • Milestone changed from Release 3.3 to Release 3.4

comment:3 Changed 6 years ago by Raquel Alvarez Banos

  • Description modified (diff)

comment:4 Changed 6 years ago by Raquel Alvarez Banos

  • Status changed from assigned to inprogress

comment:5 Changed 6 years ago by Raquel Alvarez Banos

Re #10499 Updating CMakeLists

Changeset: 76ccaa9a77d5faca0722a717ff434283bdc2b59c

comment:6 Changed 6 years ago by Raquel Alvarez Banos

Re #10499 RRFMuon base implementation

Changeset: f2b5d58f0b6290e643d022f3210697d33380d3f5

comment:7 Changed 6 years ago by Raquel Alvarez Banos

Re #10499 Updating code according to RRF paper

Changeset: ce3bd2c9c0f1bd152d16775e5f83c5c5edad7141

comment:8 Changed 6 years ago by Raquel Alvarez Banos

For tester:

  • Check the new files (.cpp, .h, unit test and documentation)
  • Check that the algorithm works for any input workspace containing two spectra
  • Check that, if the frequency is set to 0 (and also the phase), input and output spectra are the same

comment:9 Changed 6 years ago by Raquel Alvarez Banos

Re #10499 Updating unit test and documentation

Changeset: 32c859562c796ecc525d636f66fa7f1e4d9c069d

comment:10 Changed 6 years ago by Raquel Alvarez Banos

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:11 Changed 6 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:12 Changed 6 years ago by Karl Palmen

  • Status changed from verifying to verify
  • Tester Karl Palmen deleted

Cannot find branch for this ticket.

comment:13 Changed 6 years ago by Raquel Alvarez Banos

Thanks Karl. It should be ok now.

comment:14 Changed 6 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:15 Changed 6 years ago by Karl Palmen

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/10499_add_RRFMuon'

Full changeset: ​eed964f6aeb550155de165ed5a095d0201f19f40

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11341

Note: See TracTickets for help on using tickets.