mantidproject / vesuvio / 17015743774

50%

Build Type

push

github

Committed by web-flow

Commit Message

Inputs script improvements (#197)

* Remove vesuvio thickness from user inputs

- This value used to be hidden and hardcoded into the routine
- Following feedback from scientists, this change should be reverted to
how it was before

* Add system test for estimating h ratio

* Add option to chose index for H ratio

- Allow users to select mass to base hydrogen ratio from
- This change comes directly out of scientist's feedback since more
often than not they can estimate the ratio to one of the masses based
from stochiometry
- Avoids having to run the preliminary procedure so often

* Update unit test

* Catch error for missing fitting workspaces

Previously the error looked very intimidating and was not very
informative. Catching the error also allows the routine to continue
normally.

* Replace np.trapz with np.trapezoid

Due to change in numpy API

* Add comments and rename fit function

- Renamed `ansiogauss` (which should have been `anisogauss` anyway) to
`gauss2d`
- Added useful comments explaining several parameters to the user

Run Details

7 of 42 new or added lines in 4 files covered. (16.67%)

2 existing lines in 1 file now uncovered.

1405 of 2673 relevant lines covered (52.56%)

0.53 hits per line