Ticket #8615 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

WinXP LoadLotsOfFiles system tests broken

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

A snippet of the log says

FAILED TO LOAD 'C:/jenkins/workspace/is_test_winxp_develop/Data/ILLIN5_Sample_096003.nxs' WITH ERROR:

Cannot allocate 402653184 bytes of memory to load the data

This appears to be an issue with the order of loading. When loading separately in MantidPlot on XP the files do load successfully. It would appear that the by the time it gets to these files in the test that it can't allocated ~380Mb of contigous memory & falls over.

The use of XP is getting very low now so just reorder the files to avoid too much effort in mangling to code to avoid this.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Martyn Gigg

Add a list of files that are to be loaded first by LoadLotsOfFiles.

Basically avoids a problem on lower-powered XP machine that it's not worth changing the code to fix as the actual file does load in normal circumstances. Refs #8615

Changeset: 9142314bc236b95ca2691a5653e67965ade8b25d

comment:3 Changed 7 years ago by Martyn Gigg

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

Branch: bugfix/8615_loadlotsoffiles_order in the systemtests repository only.

Tester: This was only a problem on Windows XP. I suggest just looking here and seeing that the latest develop run did not fail on LoadLotsOfFiles as the previous ones have been doing. There is currently another broken test but that is being fixed elsewhere.

Last edited 7 years ago by Martyn Gigg (previous) (diff)

comment:4 Changed 7 years ago by Owen Arnold

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

comment:5 Changed 7 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8615_loadlotsoffiles_order'

Full changeset: 59bf9528be7f9f9e71ad3df7754dc7e1fc7881c5

comment:6 Changed 7 years ago by Owen Arnold

This appears to be working OK. Solutions seems to be a good one.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9459

Note: See TracTickets for help on using tickets.