Ticket #6962 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

SANS: Move SANSWideAngleCorrection to Python 2nd version

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: critical Milestone: Release 2.6
Component: Python Keywords: Maintenance
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

Correct the work properly as explained at #6529

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Keywords Maintenance added
  • Priority changed from blocker to critical

comment:2 Changed 7 years ago by Gesner Passos

Creation of SANSWideAngleCorrection python api 2

re #6962

Changeset: 1c82fa40cf6cd8bbf507800cf64975b012f8ccba

comment:3 Changed 7 years ago by Gesner Passos

Update documentation SANSWideAngleCorrection

re #6962

Changeset: 44d9cefd1b24b32df861bc55d2c40730b8376c35

comment:4 Changed 7 years ago by Gesner Passos

Some improvements to SANSWideAngleCorrection

Progress reporting, and moving toward directives of python new api.

re #6962

Changeset: 4b07e10018a1d4b95b22d6f2ca51342f2af3e30b

comment:5 Changed 7 years ago by Gesner Passos

Add unit test for SANSWideAngleCorrection

re #6962

Changeset: 4264c9b035dc4102e37a6af724e14229638d46d3

comment:6 Changed 7 years ago by Gesner Passos

Make the unit test independent of loading file data

it is required from the 'good practices' inside Mantid Team.

re #6962

Changeset: 91ed168d269b98c74dc90b1a951e0bb8b701a3dd

comment:7 Changed 7 years ago by Gesner Passos

Move documentation note to the usage section

The scientist suggested to put there. Now, that I'm aware of WIKI_USAGE keyword, I decided to put inside the usage this note.

re #6962

Changeset: 6d17b8ed4e4ff873eaefe98ba3cab1e3acb86263

comment:8 Changed 7 years ago by Gesner Passos

Adjust the equations so to not break the qtassistant

re #6962

Changeset: 2ac65a3292ee003de2b2f5815476ee6189be331e

comment:9 Changed 7 years ago by Gesner Passos

  • Status changed from new to accepted

comment:10 Changed 7 years ago by Gesner Passos

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

It does not only translated the algorithm to v2, but I also tried to fulfill the requirements for adding a unittest, adding documentation, and so on.

So the Tester is invited to:

comment:11 Changed 7 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:12 Changed 7 years ago by Nick Draper

  • Status changed from verifying to closed

Really nice documentation. Also addition of a unit test which is really appreciated. Code review is sensible, unit test passes well.

comment:13 Changed 7 years ago by Gesner Passos

Creation of SANSWideAngleCorrection python api 2

re #6962

Changeset: 1c82fa40cf6cd8bbf507800cf64975b012f8ccba

comment:14 Changed 7 years ago by Gesner Passos

Update documentation SANSWideAngleCorrection

re #6962

Changeset: 44d9cefd1b24b32df861bc55d2c40730b8376c35

comment:15 Changed 7 years ago by Gesner Passos

Some improvements to SANSWideAngleCorrection

Progress reporting, and moving toward directives of python new api.

re #6962

Changeset: 4b07e10018a1d4b95b22d6f2ca51342f2af3e30b

comment:16 Changed 7 years ago by Gesner Passos

Add unit test for SANSWideAngleCorrection

re #6962

Changeset: 4264c9b035dc4102e37a6af724e14229638d46d3

comment:17 Changed 7 years ago by Gesner Passos

Make the unit test independent of loading file data

it is required from the 'good practices' inside Mantid Team.

re #6962

Changeset: 91ed168d269b98c74dc90b1a951e0bb8b701a3dd

comment:18 Changed 7 years ago by Gesner Passos

Move documentation note to the usage section

The scientist suggested to put there. Now, that I'm aware of WIKI_USAGE keyword, I decided to put inside the usage this note.

re #6962

Changeset: 6d17b8ed4e4ff873eaefe98ba3cab1e3acb86263

comment:19 Changed 7 years ago by Gesner Passos

Adjust the equations so to not break the qtassistant

re #6962

Changeset: 2ac65a3292ee003de2b2f5815476ee6189be331e

comment:20 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:21 Changed 7 years ago by Nick Draper

  • Component changed from Framework to Python API

comment:22 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7808

Note: See TracTickets for help on using tickets.