Ticket #7783 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Use FindPeakBackground in FindPeaks

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By: #7449
Blocking: Tester: Russell Taylor

Description (last modified by Vickie Lynch) (diff)

Use peak range and background estimate in FindPeakBackground.

Change History

comment:1 Changed 7 years ago by Vickie Lynch

  • Description modified (diff)

comment:2 Changed 7 years ago by Vickie Lynch

  • Milestone changed from Backlog to Release 3.0

comment:3 Changed 7 years ago by Vickie Lynch

  • Status changed from new to inprogress

Refs #7783 use FindPeakBackground in FindPeaks

Changeset: 7bd81b7164ce395b3a64b90077f4322dd3e9abdb

comment:4 Changed 7 years ago by Vickie Lynch

Refs #7783 subtract i_min from peak boundaries

Changeset: 2a0bf22c3f2e6d721040f6e530fcbe2f1ee9c475

comment:5 Changed 7 years ago by Vickie Lynch

Refs #7783 more peaks for diamond calibration

Changeset: e52a9e7502793eb53dbc70fa5cdf31d9ab36ebe1

comment:6 Changed 7 years ago by Vickie Lynch

Refs #7783 golden data updated

Changeset: c6686fe226d1fcf2034be930f86bd8557714e9c5

comment:7 Changed 7 years ago by Vickie Lynch

Refs #7783 fix valgrid errors

Changeset: b9fe0500f0e8e46993bdbe9ab4ddd0474c85049c

comment:8 Changed 7 years ago by Peter Peterson

I'm becoming suspicious that the golden data should not have been updated.

comment:9 Changed 7 years ago by Vickie Lynch

Refs #7783 golden data updated again

Changeset: b7a2dc370e7292e955ce8055068b6616cf927d36

comment:10 Changed 7 years ago by Vickie Lynch

Refs #7783 use same background type

Changeset: d288f2934972f061aae2a5f9684b39a25681db72

comment:11 Changed 7 years ago by Vickie Lynch

Refs #7783 golden data updated again

Changeset: 541e8f68dc814580a2aee05a8efe973e3ae0029c

comment:12 Changed 7 years ago by Peter Peterson

Re #7783. Pick estimate with best chisq

Changeset: 534543ba4bdcab62f99fb75d2876b7a9b79bded7

comment:13 Changed 7 years ago by Vickie Lynch

Refs #7783 slightly increase tolerance

Changeset: 4a790266554fc0b5c7a290f04c4194cad93d4e4f

comment:14 Changed 7 years ago by Vickie Lynch

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

Calibration system test passes

comment:15 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:16 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The redundant FindPeaks::estimateBackground method needs removing. And a small one: in fitPeakHighBackground don't pass an int (4 bytes) by const-ref (8 bytes).

comment:17 Changed 7 years ago by Vickie Lynch

  • Status changed from reopened to inprogress

comment:18 Changed 7 years ago by Vickie Lynch

Refs #7783 remove estimateBackground;use int in fitPeakHighBackground

Changeset: 78ceb54275bddd3ecc4afe020e6073acc76eb35c

comment:19 Changed 7 years ago by Vickie Lynch

Revert "Refs #7783 remove estimateBackground;use int in fitPeakHighBackground"

This reverts commit 78ceb54275bddd3ecc4afe020e6073acc76eb35c.

Changeset: 189af9acc1aee05ee772aa44ffe1e23d419af1c6

comment:20 Changed 7 years ago by Vickie Lynch

Refs #7783 change to int;fit from information to debug

Changeset: 59fd67e8864c5a69c6d64778024a22c36ad4288c

comment:21 Changed 7 years ago by Vickie Lynch

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

feature/7783_FindPeaks_background branch in both mantid and SytemTests

comment:22 Changed 7 years ago by Vickie Lynch

Refs #7783 faster system test

Changeset: b42391b55136758cb04a28ec5d85295e81b5aefa

comment:23 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying

comment:24 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/7783_FindPeaks_background'

comment:25 Changed 7 years ago by Russell Taylor

Merge remote branch 'origin/feature/7783_FindPeaks_background'

comment:26 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8628

Note: See TracTickets for help on using tickets.