Ticket #9843 (reopened)

Opened 6 years ago

Last modified 5 years ago

Add CPU widget to mantid

Reported by: Peter Peterson Owned by: Martyn Gigg
Priority: major Milestone: Release 3.5
Component: Framework Keywords:
Cc: doucetm@… Blocked By:
Blocking: Tester: Alex Buts

Description

The mslice 2.0 interface has a nice widget that uses psutil to display cpu utilization and memory usage. Turn this into a QWidget that can be docked along with the workspace tree and algorithms list.

There will need to be some consideration on how to deal with whether or not psutil is installed/included.

Attachments

Mantid.png (186.7 KB) - added by Alex Buts 6 years ago.
SysMonRHEL6-Mantid.png (130.7 KB) - added by Martyn Gigg 6 years ago.
3DSetupSplashBug-Windows.png (69.6 KB) - added by Martyn Gigg 6 years ago.
SysMonRHEL6.png (120.2 KB) - added by Martyn Gigg 6 years ago.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

Add first cut at integrating S. Millers' system monitor.

At the moment it requires psutil to work. Refs #9843

Changeset: c93857459328845a14c3ea537516e3bfa0a0e00c

comment:3 Changed 6 years ago by Martyn Gigg

Add a check for the psutil package

The system monitor is only made available if this package is present. Refs #9843

Changeset: e59fb62ec8f1d28de3fe6d30383d20463bedaecd

comment:4 Changed 6 years ago by Martyn Gigg

Fix order in which psutil check is applied.

The scripting environment needs to have been started before we can check for the package. Refs #9843

Changeset: 666e9bfbb3064c58f1623b329849936cd5899c87

comment:5 Changed 6 years ago by Martyn Gigg

Install the SysMon utility to include it in the package

Refs #9843

Changeset: 04c574420718614e4d85a25ad4db05f63582f685

comment:6 Changed 6 years ago by Martyn Gigg

Don't use negative sizes in the max width settings.

Refs #9843

Changeset: f18d79b8ff43f0709bc4e5d2abe99f9cdc463cbd

comment:7 Changed 6 years ago by Martyn Gigg

Updating to SysMon v0.23

Refs #9843

Changeset: 68c021ccbf466abf4eddbc5b0caf34cd622c8579

comment:8 Changed 6 years ago by Martyn Gigg

Changes in https://github.com/mantidproject/3rdpartylibs-mac/ repo:

Bundle psutil python package

Refs #9843

Changeset: 57005fb836471c8047f85f12dce778bc26ebd8f7

comment:9 Changed 6 years ago by Martyn Gigg

comment:10 Changed 6 years ago by Martyn Gigg

Merge branch 'master' into feature/9843_integrate_sysmonitor_widget

Conflicts:

Code/Mantid/MantidPlot/CMakeLists.txt Code/Mantid/MantidPlot/src/ApplicationWindow.h Code/Mantid/MantidPlot/src/Mantid/MantidDock.cpp Code/Mantid/MantidPlot/src/Mantid/MantidUI.h

Refs #9843

Changeset: 9ea4976dd59c526ec0b6dce6aa7c3bee68e228e4

comment:11 Changed 6 years ago by Martyn Gigg

Fix bad merge conflict resolution

Refs #9843

Changeset: 5742e0719639d0f473b6092c39e00a6137162f5d

comment:12 Changed 6 years ago by Martyn Gigg

Merge branch 'master' into feature/9843_integrate_sysmonitor_widget

Conflicts:

Code/Mantid/MantidPlot/CMakeLists.txt

Refs #9843

Changeset: ec3a15b35559441583133683f47b49327a6ab4ab

comment:13 Changed 6 years ago by Martyn Gigg

Update SysMon to v0.26

Refs #9843

Changeset: bc3246d52b2f4e81210c02b73652d420aef061a9

comment:14 Changed 6 years ago by Martyn Gigg

Add psutil as package dependency for Linux packages.

Refs #9843

Changeset: 9ef4c184dab259455877c208c878fcac04c1f101

comment:15 Changed 6 years ago by Martyn Gigg

  • Owner changed from Steve Miller to Martyn Gigg
  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:16 Changed 6 years ago by Martyn Gigg

This is being verified as pull request #103.

comment:17 Changed 6 years ago by Alex Buts

  • Status changed from verify to verifying
  • Tester set to Alex Buts

comment:18 Changed 6 years ago by Alex Buts

  • Status changed from verifying to reopened
  • Resolution fixed deleted

There are some problems on RHEL plus additional annoying feature under windows, which states that Mantid uses 99% cpu when in fact it is 1% (see the image provided)

Changed 6 years ago by Alex Buts

comment:20 Changed 6 years ago by Martyn Gigg

There are some issues on RHEL6 too. See screenshot.

Changed 6 years ago by Martyn Gigg

comment:21 Changed 6 years ago by Martyn Gigg

  • Status changed from reopened to inprogress

Updated to SysMon v0.27

Refs #9843

Changeset: 4f9e81639e8219fbf3ee5384c513571ae2f0112c

comment:22 Changed 6 years ago by Martyn Gigg

Update SysMon to version 0.28

Refs #9843

Changeset: 5fa5b77c5c91ab8e29de62bc92836db7a4b89bc6

comment:23 Changed 6 years ago by Martyn Gigg

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

comment:24 Changed 6 years ago by Martyn Gigg

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:25 Changed 6 years ago by Martyn Gigg

There is still a RHEL6 issue

Changed 6 years ago by Martyn Gigg

Changed 6 years ago by Martyn Gigg

comment:26 Changed 6 years ago by Martyn Gigg

The 3DSetupSplashBug-Windows was added in error

comment:27 Changed 6 years ago by Martyn Gigg

  • Status changed from reopened to inprogress

Update SysMon to version 0.29

Refs #9843

Changeset: c15e56ff853c09ee3976a04e7a1fcb6addeda54e

comment:28 Changed 6 years ago by Martyn Gigg

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

comment:29 Changed 6 years ago by Martyn Gigg

  • Milestone changed from Release 3.3 to Release 3.4

comment:30 Changed 6 years ago by Martyn Gigg

Merge branch 'master' into feature/9843_integrate_sysmonitor_widget

Conflicts:

Code/Mantid/CMakeLists.txt

Refs #9843

Changeset: 7e39b5a4e7259278255abf723f1b3d30ae39257a

comment:31 Changed 6 years ago by Federico M Pouzols

  • Status changed from verify to verifying
  • Tester changed from Alex Buts to Federico M Pouzols

comment:32 Changed 6 years ago by Federico M Pouzols

  • Tester changed from Federico M Pouzols to Alex Buts

Sorry, my bad. Reverting to its proper state...

comment:33 Changed 6 years ago by Alex Buts

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9843_integrate_sysmonitor_widget'

Full changeset: 8efc44804a5bc9a95ac8952cb55c573569c4472b

comment:34 Changed 6 years ago by Dan Nixon

I'm getting this message on the Results Log which appears to be being printed relatively frequently (<60 seconds), making the Results Log sort of unusable as I can only scroll back for as long as whatever the interval is: ` Traceback (most recent call last):

File "/home/dan/dev-builds/msdfit-algorithm-build/bin/SysMon/sysmon.py", line 171, in constantUpdate

constantUpdateActor(self,config)

File "/home/dan/dev-builds/msdfit-algorithm-build/bin/SysMon/sysmon_tools.py", line 116, in constantUpdateActor

self.ui.cpu=np.roll(self.ui.cpu,1)

AttributeError: 'Ui_Form' object has no attribute 'cpu' `

comment:35 Changed 6 years ago by Martyn Gigg

@DanNixon has also noted that the MantidPlot docks don't remember their settings properly any more.

As this still seems to have problems that will affect everyone I'm going to revert the merge.

comment:36 Changed 6 years ago by Martyn Gigg

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:37 Changed 5 years ago by Nick Draper

  • Milestone changed from Release 3.4 to Release 3.5

Moved to R3.5 at the end of R3.4

comment:38 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10685

Note: See TracTickets for help on using tickets.