By Number of Bins odd behavior

[Michael Reuter]

The By Number of Bins (bnb) checkbox on the rebinner ParaView plugin is showing some odd behavior. There are two cases where things go funny.

1. bnb unchecked, but set to checked after applying rebinning with no axis reorientations.
2. bnb state is correct after applying rebinning with axis reorientations, but fails to cause the Bins/Step box flip without a check/uncheck cycle.

[Michael Reuter]

Refs #7849. Making sure things get/keep current state.

Changeset: c3449b224db0b0da96fb9c2d0d8380cb6f111110

[Michael Reuter]

Refs #7849. Fixing test.

Changeset: f231055f08599effc400136ca4c183d4c6b56421

[Michael Reuter]

The branch is ​here.

To test: Please try each of these cases before applying the fix. Both cases start by loading the SEQ_MDEW.nxs file into Mantid and opening the workspace in the VSI. Next, click the Rebin button.

Case 1:

• Uncheck the By Number of Bins checkbox
• Click Apply

The By Number of Bins checkbox is checked again in spite of the original state. After the fix, the original state will be remembered.

Restart the VSI between cases.

Case 2:

• Uncheck the By Number of Bins checkbox
• Switch the first axis to the T-AXIS via the drop-down box.
• Click Apply
• Check the By Number of Bins checkbox

You will notice that the Step box does not change to Bins unless you click the By Number of Bins checkbox twice more. After the fix, the first check will make things swap.

[Owen Arnold]

Merge remote-tracking branch 'origin/bugfix/7849_rebinner_nbn_behavior'

[Owen Arnold]

Nice to see someone else making use of cardinalities and actions in gmock. Now works exactly as I would expect it to.

[Stuart Campbell]

This ticket has been transferred to github issue ​8694