Ticket #8944 (closed: fixed)
Performance Investigations
Reported by: | Owen Arnold | Owned by: | Owen Arnold |
---|---|---|---|
Priority: | blocker | Milestone: | Release 3.2 |
Component: | Reflectometry | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Martyn Gigg |
Description
Aim here is three fold.
- Firstly, identify or generate some standard reductions scripts to use as benchmarks
- If this could be wrapped up in a performance test(s), that would be great.
- Compile a list of performance bottlenecks.
Only after this can we make targeted performance improvements. Each enhancement occurring as a new ticket.
1-Week effort
Change History
comment:3 Changed 6 years ago by Owen Arnold
1) Jean is performing the critical benchmarking for the SNS #8929
2) The new c++ workflow algorithms are much faster than their older python counterparts. This has been reported by the users, who are now happy with the speed. The main bottleneck is in IO file loading. There are two aspects here.
comment:4 Changed 6 years ago by Owen Arnold
- Status changed from assigned to verify
- Resolution set to fixed
No code changes made.
comment:5 Changed 6 years ago by Karl Palmen
- Status changed from verify to verifying
- Tester set to Karl Palmen
comment:6 follow-up: ↓ 7 Changed 6 years ago by Karl Palmen
- Status changed from verifying to reopened
- Resolution fixed deleted
This ticket is a blocker, but the ticket #8929 for the benchmarking is not a blocker. This ticket would pass if the priorities were changed appropriately.
comment:7 in reply to: ↑ 6 Changed 6 years ago by Owen Arnold
- Status changed from reopened to verify
- Resolution set to fixed
Replying to Karl Palmen:
This ticket is a blocker, but the ticket #8929 for the benchmarking is not a blocker. This ticket would pass if the priorities were changed appropriately.
This ticket is not blocking anything. I'm putting it back into the pool.
comment:8 Changed 6 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester changed from Karl Palmen to Martyn Gigg
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9787
Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state