Ticket #3533 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

EQSANS: compute Q resolution taking into account TOF resolution

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

The TOF resolution has its own wavelength dependence. Use fit provided by instrument scientist to implement Q resolution for EQSANS.

Change History

comment:1 Changed 9 years ago by Mathieu Doucet

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Mathieu Doucet

In [13834]:

Implemented child class of TOFSANSResolution for EQSANS. Re #3533

comment:3 Changed 9 years ago by Mathieu Doucet

In [13835]:

Use new EQSANSResolution algorithm. Re #3533

comment:4 Changed 9 years ago by Mathieu Doucet

In [13838]:

Cleanup. Re #3533

comment:5 Changed 9 years ago by Mathieu Doucet

In [13839]:

Cleanup. Re #3533

comment:6 Changed 9 years ago by Mathieu Doucet

In [13845]:

Fix warning Re #3533

comment:7 Changed 9 years ago by Russell Taylor

There's a problem here that will need sorting out - algorithm packages are not allowed to depend on one another, i.e. WorkflowAlgorithms cannot depend on Algorithms. Apart from trying to avoid dependency spaghetti, it breaks the Mac install.

comment:8 Changed 9 years ago by Mathieu Doucet

In [13983]:

Move EQSANSResolution to Algorithms. Re #3533

comment:9 Changed 9 years ago by Russell Taylor

In [13984]:

Remove package dependency. Re #3533.

comment:10 Changed 9 years ago by Mathieu Doucet

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

comment:11 Changed 9 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:12 Changed 9 years ago by Peter Peterson

  • Status changed from verifying to closed

Looks good

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4380

Note: See TracTickets for help on using tickets.