Ticket #8944 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state

comment:2 Changed 7 years ago by Owen Arnold

  • Component changed from Framework to Reflectometry

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.

  • We have reduced the loading of files in the GUI. For example, we attempt to extract run numbers from titles where possible. #9167
  • There is an already a ticket to look at HDF5 instead of Nexus APIs for IO. This is the only sensible route to IO speedup. #9450
Last edited 6 years ago by Owen Arnold (previous) (diff)

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:9 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9787

Note: See TracTickets for help on using tickets.