Ticket #3244 (closed: fixed)
Make LoadEventPreNeXus faster
Reported by: | Peter Peterson | Owned by: | Janik Zikovsky |
---|---|---|---|
Priority: | major | Milestone: | Iteration 30 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Peter Peterson |
Description
This is a continuation of #3224.
Profiling still says that the majority of the time spent is in processing the list rather than reading from disk by at least a factor of 100. The next big target is finding a better way of allocating memory in the individual event lists (via reserve) rather than just using the defaults and letting vector re-allocate memory as necessary.
Change History
comment:1 Changed 9 years ago by Janik Zikovsky
- Owner set to Janik Zikovsky
- Status changed from new to accepted
comment:8 Changed 9 years ago by Janik Zikovsky
- Status changed from accepted to verify
- Resolution set to fixed
In [13700]:
comment:9 Changed 9 years ago by Peter Peterson
- Status changed from verify to verifying
- Tester set to Peter Peterson
comment:10 Changed 9 years ago by Peter Peterson
- Status changed from verifying to closed
It is faster.
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4091
Note: See
TracTickets for help on using
tickets.
I think the theory on how to do it is to split up the file in N small (but not too small) chunks and load them into separate EventWorkspaces which we will then merge into one big one.
Will probably require moving quite a bit of code around in the algorithm.