Ticket #8101 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Add a system test for powder diffractometer profile calibration algorithms

Reported by: Wenduo Zhou Owned by: Wenduo Zhou
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: petersonpf@… Blocked By: #7880
Blocking: Tester: Samuel Jackson

Description

Add a system test for the workflow algorithms to calibrate powder diffractometer's profile parameters. It should cover algorithm ExaminePowderDiffProfile and the algorithm to refine profile parameters.

Change History

comment:1 Changed 7 years ago by Wenduo Zhou

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Wenduo Zhou

Removed 1 unused property. Refs #8101.

Changeset: 14aa1e9fb2f5f7f757439c862de1fbafd764ebc6

comment:3 Changed 7 years ago by Wenduo Zhou

Initialize system test. Refs #8101.

Changeset: 9d9d64c3306e1e50c0fa0c8338b3931d200e936f

comment:4 Changed 7 years ago by Wenduo Zhou

Fixed some minor bugs. Refs #8101.

Changeset: 0bc4b72854cca4b20720ad9ece91a0c4b615729e

comment:5 Changed 7 years ago by Wenduo Zhou

Added test data. Refs #8101.

Changeset: c615652682a93ebc59f86a8d0cf3f731a62c7e39

comment:6 Changed 7 years ago by Wenduo Zhou

Added the golden file. Refs #8101.

Changeset: 1f876dae313584c5167226c2fcfe92b2a4c410ba

comment:7 Changed 7 years ago by Wenduo Zhou

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

For Testers

It is easy to test this ticket. As long as the new system test is tested by all build servers and passes, the task of this ticket is finished.

comment:8 Changed 7 years ago by Wenduo Zhou

Added tolerance to fix Mac build. Refs #8101.

Changeset: edb8c34614533e3d25fa0f7631c265ebf8056db6

comment:9 Changed 7 years ago by Samuel Jackson

  • Status changed from verify to verifying
  • Tester set to Samuel Jackson

comment:10 Changed 7 years ago by Samuel Jackson

  • Status changed from verifying to reopened
  • Resolution fixed deleted

I can't seem to find the branch for commit 14aa1e9fb2f5f7f757439c862de1fbafd764ebc6. Is it that it just wasn't pushed to GitHub?

comment:11 Changed 7 years ago by Wenduo Zhou

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

I forgot to commit that branch. Thanks for letting me know. It is in the repository now.

comment:12 Changed 7 years ago by Samuel Jackson

  • Status changed from verify to verifying

comment:13 Changed 7 years ago by Samuel Jackson

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8101_lebail_systemtest'

Full changeset: bedea43afbb404773c27febafa43f66494c5afd4

comment:14 Changed 7 years ago by Samuel Jackson

Merge remote-tracking branch 'origin/feature/8101_lebail_systemtest'

Full changeset: e31dda3d1bdd771b5ef97240129d42a80e3353bc

comment:15 Changed 7 years ago by Samuel Jackson

System test passes and code looks good.

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8946

Note: See TracTickets for help on using tickets.