Ticket #10617 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Bug in NeXus when loading large event files

Reported by: Martyn Gigg Owned by: Stuart Campbell
Priority: critical Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description

The bug has been fixed for sometime: ​http://trac.nexusformat.org/code/ticket/349. The fix is not in our current packages for RHEL/Fedora though.

If we mandate 4.3.2 everywhere and update the RHEL/Fedora packages we will also need to recompile the Windows/OS X libraries. The other solution is to only mandate >=4.3.1 then everywhere should be happy.

Change History

comment:1 Changed 6 years ago by Martyn Gigg

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Stuart Campbell

  • Status changed from assigned to inprogress

The NeXus rpms have been rebuilt for RHEL/Fedora and are available in the Mantid COPR ​https://copr.fedoraproject.org/coprs/mantid/mantid/ and the RPMs directly accessible at ​http://copr-be.cloud.fedoraproject.org/results/mantid/mantid/

If they are needed at ISIS, then someone with access will need to copy them (my fedid has been disabled again).

comment:3 Changed 6 years ago by Martyn Gigg

Should we update the CMake & package requirements to mandate a minimum version of 4.3.1?

comment:4 Changed 6 years ago by Nick Draper

Is there any progress on this? Can we get it into this release?

comment:5 Changed 6 years ago by Martyn Gigg

Update minimum required NeXus version.

Refs #10617

Changeset: b63e9f2595734f544990d6979188a20d7d6a2a2f

comment:6 Changed 6 years ago by Martyn Gigg

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

This is being verified as ​pull request #123.

comment:7 Changed 6 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:8 Changed 6 years ago by Stuart Campbell

  • Status changed from verifying to closed

Merge pull request #123 from mantidproject/feature/10617_update_min_nxs_version

Update minimum required NeXus version.

Full changeset: ​7b0bf4cd2f8273fa8d3af00e0c413706af18e079

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11459

Note: See TracTickets for help on using tickets.