Ticket #3217 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

PerformanceTests: Notify when performance drops

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 30
Component: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description


Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Milestone changed from Iteration 29 to Iteration 30

Moved my tickets that won't get finished by iteration 29

comment:2 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:3 Changed 9 years ago by Janik Zikovsky

(In [12923]) Added a python script to check the performance test results relative to previous performance, and to make a warning if a test slows down. Refs #3217. P.S. This doesn't affect the either iteration 29 or iteration 30 since it is just a tool for the build jobs.

comment:4 Changed 9 years ago by Janik Zikovsky

(In [12925]) Refs #3217: Niceties in printing out the check_performance report.

comment:5 Changed 9 years ago by Janik Zikovsky

(In [12935]) Corrected Q in MakeDiffractionEventWorkspace (to not have 2*pi and to be q = ki-kf, it was kf-ki). Added a Paraview plugin, PeaksReader, to load .peaks file to visualize them. Needed to add a link to the Crystal library. Refs #3232. Added a vtkPeakMarkerFactory to render single-crystal peaks (as points). Not totally finished. Refs #3232. Made vtkDataSetFactory::initialise() take a Workspace_sptr instead of a IMDWorkspace_sptr. This allowed the visualization of a PeaksWorkspace. Still needs tests. Made the MultiDimensionalDbPresenter::getMesh() method return a vtkDataSet* (instead of vtkUnstructuredGrid*) to be more general. Attempted to speed up/parallelize vtkThresholdingHexahedronFactory without much improvement so far. Refs #3271. Made a MockObjects.hh file to reuse mock objects code in VatesAPITest. Fixes #3291. Sped up setUp of EventListTestPerformance; also InstrumentRayTracerTestPerformance. Refs #3217.

comment:6 Changed 9 years ago by Janik Zikovsky

(In [12940]) Refs #3217: Intentionally slowing down a performance test in order to trigger a nastygram from Jenkins.

comment:7 Changed 9 years ago by Janik Zikovsky

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

(In [12943]) Fixes #3217: Yes, the emails from the build server work.

comment:8 Changed 9 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:9 Changed 9 years ago by Russell Taylor

  • Status changed from verifying to closed

I expect we've all received at least one of these emails over the past few months....

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4064

Note: See TracTickets for help on using tickets.