Update reduction wrapper according to current web services state

[Alex Buts]

When writing reduction wrapper, I was expecting web service to be written as a factory with reduction being a subscriber.

Web service is currently done in a bit different way, which causes inefficiencies & inconveniences in reduction wrapper. In addition to that, it is clear now how multiple runs are handled and what are user requests to do that from web and desktop -- this should be reconciled and satisfied.

This work would allow to test new reduction in all instances on non-event mode instruments and obtain an early feedback on this reduction. This is why it goes before #10684.

[Alex Buts]

Re #10958 Initial commit. (Nothing working yest)

Changeset: 4235123db5ff8ec035e9d37798a14fb0c9f9c044

[Alex Buts]

Re #10958 Intermediate checkout

Changeset: 5ae5c459ad25d6f759e52a0ac24010f22ec61bcb

[Alex Buts]

Re #10958 ReductionWrapper modified to run in web-like way

Changeset: e0e2f49a8034dace0b3d8bf6ddd131e854153689

[Alex Buts]

Re #10958 One more system test for MARI when one tries to run it

like from the web service. Changes to Merlin and Mari reduction to support new ReductionWrapper

Changeset: 4e4224772dd575847b585c81726626200409ee55

[Alex Buts]

Re #10958 Fixed output workspace name

Changeset: eabee4d50ce60fa39d4090f928af4b346d3e0fa9

[Alex Buts]

Re #10958 Modified ISIS system tests to new interface

Changeset: 6ad03d483667cc9905a71f0bb6cc23f8826e4e52

[Alex Buts]

Re #10958 minor comments

Changeset: dcf85543a770979c072a66cb1a6bb4249470b5a2

[Alex Buts]

Re #10958 fixed reduce return in case of multiple workspaces

(multirep mode)

Changeset: 485df58b0bc6d5ae7c8d9abc7789ad32c09cd131

[Alex Buts]

Re #10958 Minor comments

Changeset: 0dfb5723a8fceb108e5057e594a6eb8d78d16f26

[Alex Buts]

Re #10958 Fixed separate/common monitor loading

Changeset: 15c07d12ea69c23bacc9e42347cb7600f90d08b1

[Alex Buts]

Re #10958 Minor cosmetic changes to system tests related

to changes in Mantid branch

Changeset: 316f3de0755a1a12e8e21c10129bebe5924efcce

[Alex Buts]

Re #10958 Validate method in reduction wrapper

Changeset: 3c5c1ffd429884a722c14a10c6334c092752ffba

[Alex Buts]

Re #10958 System, test to validate method in reduction wrapper

Changeset: ae7cbab078f5da01c8e354bc91832236202a4ca5

[Alex Buts]

Re #10958 Fixed "Wait for file" mode

Changeset: e16d365d483e2f6ee8d62c9b5d8479d1637c9ca2

[Alex Buts]

Re #10958 Fixing system tests

and adding comments on why some a failing and what can be done to avoid it

Changeset: c13e458d71b88af71fee9793cec43dad67581664

[Alex Buts]

Re #10958 Just formatting

Changeset: bf0a58de45011d524b3599001eaec0385cab5b09

[Alex Buts]

Re #10958 Separated sum_runs and sample_run

This allows better separation of cases when runs are summed or considered one-by one

Changeset: 58103f20c1cc541389b06345316c2be00e0b4cbc

[Alex Buts]

Re #10958 Introducing run_list

Changeset: 08c98d0903a6b5f57aa48a9915282ad8eeeade2b

[Alex Buts]

Re #10958 Minor comments

Changeset: 92179172f9213793ce2f5b0d4bc10dfc13958426

[Alex Buts]

Re #10958 Bug in summation and better results validator

Changeset: 56919cfbf951090578d0f617d66ce2feb9eaf262

[Alex Buts]

Re #10958 Enhanced validation procedure

Changeset: 301b35b94dc85414041cdd798b7224eb5e89df3b

[Alex Buts]

Re #10958 Minor pylint warnings

Changeset: 314f3b0e873ddb7dd155ca35247506222e56fbff

[Alex Buts]

Re #10958 Pylint warnings

Changeset: 7d734cc700f5dd8633849b7eb57c03ae744ed272

[Alex Buts]

Re #10958 Fixing error caused by following pylint suggestions

This pylint is not harmless at all

Changeset: a562cd23699f5bebafab1859aafdd64f5e6b4ff1

[Alex Buts]

Re #10958 Minor bugs in validate_result ReductionWrapper script

Changeset: 7ca2f6388fe302a5e374f2492729df40014e4ff9

[Alex Buts]

Changes are both in Mantid and System test directory as I've written&modified number of tests to validate the changes.

The changes in the ticket allow to take most of current state of the Autoreduction service.

I've also separated RunDescriptor and summation class, which should allow to sum runs provided as list of run numbers in autoreduction. It is not done very nicely, but should work. I will refactor this change later, which would also allow data accumulation.

Minimal testing relies to all system test passing.

If tester decides to proceed with code review, please talk to me -- I am not very satisfied with current changes code, but have to finish ticket #10684 first.

[Alex Buts]

system test have problem they did not have before

[Alex Buts]

Re #10958 Fixing errors, induced by pylint

Changeset: 9700712e123326bb87b2eaa8fbb798515a59dce7

[Ian Bush]

System tests and unit tests all run ok, and changes look sensible, so accepting.

[Ian Bush]

Merge remote-tracking branch 'origin/feature/10958_UpdatedReductionWrapper'

Full changeset: ​9769182cc72611f66b59e8c239ee96a9155863cc

[Ian Bush]

Merge remote-tracking branch 'origin/feature/10958_UpdatedReductionWrapper'

Full changeset: ​9769182cc72611f66b59e8c239ee96a9155863cc

[Stuart Campbell]

This ticket has been transferred to github issue ​11797