Ticket #11022 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Fix failing PoldiAnalyseResiduals doctest

Reported by: Michael Wedel Owned by: Michael Wedel
Priority: major Milestone: Release 3.4
Component: Diffraction Keywords: POLDI
Cc: Blocked By:
Blocking: Tester: Federico M Pouzols

Description (last modified by Michael Wedel) (diff)

In tickets #10263 and #10774 some POLDI algorithms changed and somehow the changes in the documentation overlapped and after merging into master, the doctest for PoldiAnalyseResiduals failed because of a missing output parameter. I'm going to add the parameter.

Change History

comment:1 Changed 6 years ago by Michael Wedel

  • Milestone changed from Backlog to Release 3.4

comment:2 Changed 6 years ago by Michael Wedel

  • Description modified (diff)
  • Summary changed from Fix failing PoldiFitPeaks2D doctest to Fix failing PoldiAnalyseResiduals doctest

comment:3 Changed 6 years ago by Michael Wedel

Refs #11022. Fixing PoldiAnalyseResiduals doctest

Changeset: 515e5bdc8c7da8d3a423ee1bed1c2510c8190f5f

comment:4 Changed 6 years ago by Michael Wedel

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

This is being verified as ​pull request #175.

comment:5 Changed 6 years ago by Michael Wedel

Testing information

Make sure the doctest of PoldiAnalyseResiduals passes.

comment:6 Changed 6 years ago by Federico M Pouzols

  • Status changed from verify to verifying
  • Tester set to Federico M Pouzols

comment:7 Changed 6 years ago by Federico M Pouzols

Doc tests passed locally.

comment:8 Changed 6 years ago by Federico M Pouzols

Merge pull request #175 from mantidproject/11022_fix_poldi_analys_residuals_doctest

Refs #11022. Fixing PoldiAnalyseResiduals doctest

Changeset: 1ef77d71d9f4b2e8f2dbc700b918f8c7c51a767b

comment:9 Changed 6 years ago by Federico M Pouzols

  • Status changed from verifying to closed

Merge pull request #175 from mantidproject/11022_fix_poldi_analys_residuals_doctest

Refs #11022. Fixing PoldiAnalyseResiduals doctest

Full changeset: ​1ef77d71d9f4b2e8f2dbc700b918f8c7c51a767b

comment:10 Changed 6 years ago by Federico M Pouzols

Merge pull request #175 from mantidproject/11022_fix_poldi_analys_residuals_doctest

Refs #11022. Fixing PoldiAnalyseResiduals doctest

Changeset: 8d29d3d3db8850690a023152da82423e7e788454

comment:11 Changed 6 years ago by Federico M Pouzols

Merge pull request #175 from mantidproject/11022_fix_poldi_analys_residuals_doctest

Refs #11022. Fixing PoldiAnalyseResiduals doctest

Full changeset: ​1ef77d71d9f4b2e8f2dbc700b918f8c7c51a767b

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11861

Note: See TracTickets for help on using tickets.