search

mantidproject / mantid

Main repository for Mantid code
https://www.mantidproject.org
GNU General Public License v3.0
209 stars 122 forks source link

Investigate size of instrument definition files #33797

Open sf1919 opened 2 years ago

sf1919 commented 2 years ago

Describe the outcome that is desired. An investigation is required to see if the Instrument Definition Files (IDF) can be made smaller as this may improve performance of Mantid for users.

Does the outcome relate directly to a problem? Please describe. Whilst investigating memory leak problems (#33662 ) we noted that instrument definition files seemed to take up a signficant portion of the active memory on Windows. In particular the WISH definition file seemed very large. To test this we did the following:

  1. Opened recent nightly of Mantid (v6.3.20220426.1540)
  2. Using Details tab in Task Manager noted the current active memory being used by python executable (in my case 192,420K)
  3. Used LoadEmptyInstrument algorithm withInstrumentName set to WISH and outputworkspace name of your choice.
  4. Checked the active memory being used by pyton (now 1,232,952K for me).

It is likely that all IDF files are larger than they need to be so a general investigation would be useful.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had activity in 6 months. It will be closed in 7 days if no further activity occurs. Allowing issues to close as stale helps us filter out issues which can wait for future development time. All issues closed by stale bot act like normal issues; they can be searched for, commented on or reopened at any point. If you'd like a closed stale issue to be considered, feel free to either re-open the issue directly or contact a developer. To extend the lifetime of an issue please comment below, it helps us see that this is still affecting you and you want it fixed in the near-future. Extending the lifetime of an issue may cause the development team to prioritise it over other issues, which may be closed as stale instead.

sf1919 commented 1 year ago

@thomashampson one for the next sprint pehaps?