Ticket #6323 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Fix threading bug in FermiChopperModel

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

The TobyFitResolution system tests fails occasionally with this error:

ResolutionConvolvedCrossSection-[Error] ResolutionConvolvedCrossSection: FermiChopperModel::regimeFactor - gamma is greater than 4! Behaviour is undefined. Value=inf
ResolutionConvolvedCrossSection-[Error] Component values: chopper radius=0.049,deltaT=6.22545e-06,slitRadius=1.3,Ei=0,omega=3769.91,slitThickness=0.0023

The Ei value should be 300 so there is a threading issue when the FermiChopperModel retrieves the Ei.

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

Fix thread-safety issue in Cache class Refs #6323

The Cache class was not fully threadsafe. The PARALLEL sections have been replaced by a per-object FastMutex to ensure thread safety. Hopefully this won't have a huge effect on performance.

Changeset: 64d4e11e837adf99eca6c78a7f45150afcefea97

comment:3 Changed 8 years ago by Martyn Gigg

Fix return statment in coy constructor. Refs #6323

gcc let me get away with that for some reason...

Changeset: 150b9c740b7c87482b04b0d72e1f84dc2958f7c2

comment:4 Changed 8 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:5 Changed 8 years ago by Martyn Gigg

Fix thread-safety issue in Cache class Refs #6323

The Cache class was not fully threadsafe. The PARALLEL sections have been replaced by a per-object FastMutex to ensure thread safety. Hopefully this won't have a huge effect on performance.

Changeset: 2fb3881c20311ba52af65d77b8fef712f20c2661

comment:6 Changed 8 years ago by Martyn Gigg

Fix return statment in coy constructor. Refs #6323

gcc let me get away with that for some reason...

Changeset: 2f5b378603351f9f0eaaf037a1f42be96fea121b

comment:7 Changed 8 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:8 Changed 8 years ago by Owen Arnold

  • Status changed from verifying to closed

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7169

Note: See TracTickets for help on using tickets.