Ticket #7824 (closed: fixed)
Flexible Center Tolerance value
Reported by: | Gesner Passos | Owned by: | Gesner Passos |
---|---|---|---|
Priority: | major | Milestone: | Release 3.0 |
Component: | SANS | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Jay Rainey |
Description
Would it be possible to add another field to the beam centre finder part of the geometry tab which would allow us to change the convergence tolerance? Obviously it could default to whatever values it uses now
Change History
comment:3 Changed 7 years ago by Gesner Passos
Rearrange sans function to allow changing the tolerance Seek centre
Inside ISISCommandInterface.py, change the FindBeamCentre in order to allow providing a new value for tolerance. It now, uses the previous values as the default input.
re #7824
Changeset: ec4c1f0b5c24e9c58728e447659e528fa968c007
comment:4 Changed 7 years ago by Gesner Passos
Export the tolerance option to GUI SANS interface
re #7824
Changeset: aa2c4c6813bdec6306caa719ad53585374324aee
comment:5 Changed 7 years ago by Gesner Passos
- Status changed from inprogress to verify
- Resolution set to fixed
Tester:
- open the SANS interface
- change the instrument to LOQ
- Load the user file: systemtests/Data/LOQ/MASK.094AA
- Load the sample file: systemtests/Data/LOQ/LOQ54431.raw
- Change to the Geometry Tab
- check that there is a entry available for tolerance
- Run the auto search.
- It will take 7 iteractions to convert.
- Put the values of the centre back: 324.64, 327.92
- Change the tolerance to a bigger value: For example 5.0
- If you put 5.0, the auto-search will stop on the iteration 5.
comment:6 Changed 7 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:7 Changed 7 years ago by Jay Rainey
- Status changed from verifying to reopened
- Resolution fixed deleted
If values are input that are negative, or words then the default value it used. That's great, but the user isn't told about it.
comment:8 Changed 7 years ago by Owen Arnold
- Status changed from reopened to inprogress
refs #7824. Make regex more flexible.
Effectively introduce an OR into the regex. This allows it to process the newer form of the HKL coordinate transform labels. Should also still work with the old ones.
Changeset: b9f3cd99ee97baeb85966022add3ad3caee3faa6
Edit - Should have gone against ticket 7827
comment:9 Changed 7 years ago by Owen Arnold
refs #7824. Correct Regex.
Changeset: 7f2fb4fe73d6264d66bb5c17a8a4ba01539488e7
Edit - Should have gone against ticket 7827
comment:10 Changed 7 years ago by Gesner Passos
Alert user for invalid input for Tolerance
re #7824
Changeset: cf3c982a3d22305604e6141baa9319d5d83eb013
comment:11 Changed 7 years ago by Gesner Passos
- Status changed from inprogress to verify
- Resolution set to fixed
comment:13 Changed 7 years ago by Jay Rainey
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/7824_seek_centre'
comment:14 Changed 7 years ago by Jay Rainey
No longer accepts negative inputs and valid error message is produced if they are provided.
comment:15 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8669
Currently, the value that you are talking about is 0.1251/1000