Ticket #7289 (closed: fixed)
NeXus API issue with long event lists
| Reported by: | Stuart Campbell | Owned by: | Martyn Gigg |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.6 |
| Component: | Framework | Keywords: | patchcandidate |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Nick Draper |
Description (last modified by Martyn Gigg) (diff)
There is an issue we have seen with large event lists, where large means that the number of events is greater than the size of an int on that platform. This issue has been fixed upstream (see http://trac.nexusformat.org/code/ticket/349).
Although none of the code in mantid will need to be updated, the third party libs and rpm dependencies will need updating.
Change History
comment:2 Changed 7 years ago by Stuart Campbell
Updated NAPI to 4.3.2(ish) refs #7289
Changeset: eba9b75840aa5eb109ac2a5415039ea657c38210
comment:6 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5.4 to Release 2.6
Returned to Release 2.6 as patch release 2.5.4 will now not happen.
comment:8 Changed 7 years ago by Martyn Gigg
- Status changed from new to inprogress
Update napi.h to match that of current tip of NeXus 4.3 branch
This is revision [1901] Refs #7289
Changeset: f790f816dc2c4ef5868189b87cb6edb65a755538
comment:9 Changed 7 years ago by Martyn Gigg
Update Win64 Nexus libs to current tip of NeXus 4.3 branch
This is revision [1901] Refs #7289
Changeset: 0d1bbe1bd59f1b4e0c3f4d640ece7a6a0e35bc78
comment:10 Changed 7 years ago by Martyn Gigg
Update win32 NeXus libs to current tip of 4.3 branch
This is revision [1901] Refs #7289
Changeset: 26c7aab94bc2f3aad463e6f07cb5b1526691f2cd
comment:11 Changed 7 years ago by Martyn Gigg
Correct linking of Win64 NeXus to HDF libraries. Refs #7289
Changeset: 80dc570a491c0921b02540f79295c36846bae8be
comment:12 Changed 7 years ago by Martyn Gigg
Fix linking of NeXus to HDF. Refs #7289
Changeset: ee127c093ddf28c4dd37186ed6a39fc0d518f875
comment:13 Changed 7 years ago by Martyn Gigg
Update NeXus libs to current tip of 4.3 branch
This is revision 1901. Refs #7289
Changeset: 75f4475ec3a302ce707d0d71909e1922888ff924
comment:14 Changed 7 years ago by Martyn Gigg
- Owner changed from Stuart Campbell to Martyn Gigg
- Status changed from inprogress to verify
- Resolution set to fixed
- Description modified (diff)
No branch or code to merge. Everything is in third party. All of the tests should certainly be passing and we should now be able to load nexus files with large event lists in them.
comment:15 Changed 7 years ago by Nick Draper
- Status changed from verify to verifying
- Tester set to Nick Draper
comment:16 Changed 7 years ago by Nick Draper
- Status changed from verifying to closed
Accepted as this has gone into the 3rd party repos and will be shipped anyway. All unit and system tests are passing, so the file loading is fine.
It is not clear if this has actually resolved the loading large files, but it should. Stuart to retest when he is available.
comment:17 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8135
