Ticket #7824 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Gesner Passos

Currently, the value that you are talking about is 0.1251/1000

 if abs(XSTEP) < 0.1251/1000. and abs(YSTEP) < 0.1251/1000. :
  # this is the success criteria, we've close enough to the center
  centre.logger.notice("Converged - check if stuck in local minimum!")
  break

comment:2 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

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.

Last edited 7 years ago by Jay Rainey (previous) (diff)

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

Last edited 7 years ago by Owen Arnold (previous) (diff)

comment:9 Changed 7 years ago by Owen Arnold

refs #7824. Correct Regex.

Changeset: 7f2fb4fe73d6264d66bb5c17a8a4ba01539488e7

Edit - Should have gone against ticket 7827

Last edited 7 years ago by Owen Arnold (previous) (diff)

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:12 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying

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

Note: See TracTickets for help on using tickets.