Ticket #8588 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Python Algorithm Decorator

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 3.1
Component: Tools Keywords:
Cc: Blocked By: #8371
Blocking: Tester: Peter Parker

Description

Allow the construction of easy to read and concise python testing around complex workflow algorithms. Uses the builder pattern in which construction is deferred to a building/decorating type, which exposes all constructor arguments via setters. A neat feature of this is that we can automatically build the setters on the fly using introspection into the algorithm. See ReflectometryReductionOneTest.py for example usage.

This has already been implemented as part of #8371. But it would be generally useful, and I am therefore going to move it to a separate module.

Change History

comment:1 Changed 7 years ago by Owen Arnold

  • Milestone changed from Backlog to Release 3.1

comment:2 Changed 7 years ago by Owen Arnold

  • Status changed from new to inprogress

refs #8588. move algorithm_decorator into own module.

Changeset: 56bf758b95c99b66062370349ff4f33ebe288638

comment:3 Changed 7 years ago by Owen Arnold

Note to self. These changes are built upon branch of #8371. Need to rebase onto master after #8371 has been merged into master before continuing. Other than that, everything should just work.

comment:4 Changed 7 years ago by Owen Arnold

refs #8588. move algorithm_decorator into own module.

Changeset: c1e1a2f9db02d2c984c6c6e8be102259b90ffb5a

comment:5 Changed 7 years ago by Owen Arnold

Tester:

If ReflectometeryReductionOneTest.py in PythonAlgorithms is still running then this has worked.

comment:6 Changed 7 years ago by Owen Arnold

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

comment:7 Changed 7 years ago by Peter Parker

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

comment:8 Changed 7 years ago by Peter Parker

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8588_algorithm_decorator'

Full changeset: ​585b0419bfcc6d72b3ce79e21b1a77d67451d29d

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9432

Note: See TracTickets for help on using tickets.