Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mantid become sensitive to network conditions #12425

Closed
abuts opened this issue Apr 17, 2015 · 4 comments
Closed

Mantid become sensitive to network conditions #12425

abuts opened this issue Apr 17, 2015 · 4 comments
Assignees
@cailafinn
Labels
Framework Issues and pull requests related to components in the Framework ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Milestone
Release 5.1 S3

Comments

@abuts
Copy link
Member

abuts commented Apr 17, 2015

This issue was originally TRAC 11587

A problems were observed when ISIS network have problems. In particular, the archive did not work well.

Mantid become irresponsible, random crashes were observed on Windows and RHEL7, (I've seen 2 on windows and 1 on RHEL7 during 30 min attempted work) when no crashes, user sometimes was waiting for almost a minute looking at dimmed Mantid window when Mantid was trying to find a file on network or load it from network folders.

Pretty grim experience and users blame Manitd.
@abuts abuts added the Framework Issues and pull requests related to components in the Framework label Jun 3, 2015
@gemmaguest
Copy link
Member

Mantid can also be very slow to start if the archive is slow. It seems to get stuck for ages going through the recent files list.

martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 5, 2018
@martyngigg
Remove check for recent files existence on startup
9149d3e 
For files on slow mounted network drives this can lead
to long lag times when starting MantidPlot. This leaves
the check until if the file is used at any point.

Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 5, 2018
@martyngigg
Elide long recent filename items
489b2be 
Adds a tooltip with the full text so this is still retrievable.
Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 5, 2018
@martyngigg
Add a release note regarding recent files change.
faa5ee8 
Refs mantidproject#12425
@martyngigg martyngigg mentioned this issue Sep 5, 2018
Merged
martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 5, 2018
@martyngigg
Elide long recent filename items
685c584 
Adds a tooltip with the full text so this is still retrievable.
Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 5, 2018
@martyngigg
Add a release note regarding recent files change.
777c862 
Refs mantidproject#12425
@gemmaguest
Copy link
Member

Following PR #23443, I'm still getting slow startup, but now in different places. If I have files in datasearch.directories then it's slow here. Otherwise, it's slow when creating a QFileDialog here.

martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 17, 2018
@martyngigg
Convert ConfigService paths to absolute on demand
d338c6b 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs mantidproject#12425

Add comment detailing absolute path conversion
@martyngigg martyngigg mentioned this issue Sep 17, 2018
Closed
martyngigg added a commit to martyngigg/mantid that referenced this issue Sep 18, 2018
@martyngigg
Convert ConfigService paths to absolute on demand
35f35c7 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Aug 5, 2019
@martyngigg
Convert ConfigService paths to absolute on demand
f51c92e 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Aug 6, 2019
@martyngigg
Convert ConfigService paths to absolute on demand
4e9fa68 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs mantidproject#12425
martyngigg added a commit to martyngigg/mantid that referenced this issue Nov 3, 2019
@martyngigg
Convert ConfigService paths to absolute on demand
d97d56f 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs mantidproject#12425
@martyngigg martyngigg added ISIS Team: Diffraction Issue and pull requests managed by the Diffraction subteam at ISIS Maintenance Unassigned issues to be addressed in the next maintenance period. labels Nov 20, 2019
@martyngigg martyngigg added this to the Release 4.3 S1 milestone Nov 20, 2019
@martyngigg martyngigg removed this from the Release 4.3 S1 milestone Dec 11, 2019
@martyngigg martyngigg added ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS and removed ISIS Team: Diffraction Issue and pull requests managed by the Diffraction subteam at ISIS labels Apr 22, 2020
@martyngigg martyngigg added this to the Release 5.1 S2 milestone May 21, 2020
@martyngigg martyngigg removed the Maintenance Unassigned issues to be addressed in the next maintenance period. label May 21, 2020
@martyngigg martyngigg removed their assignment May 21, 2020
@DanielMurphy22 DanielMurphy22 mentioned this issue May 26, 2020
Closed
@cailafinn cailafinn self-assigned this May 26, 2020
martyngigg added a commit that referenced this issue May 26, 2020
@martyngigg
Convert ConfigService paths to absolute on demand
0e1346b 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs #12425
@gvardany
Copy link
Contributor

We also observed problems when the data are on the network mount. In case of intermittent connection interruption, Mantid freezes and you have to kill the process. The most annoying is that it gets stuck in some undetermined state, so you can not start it again even after kill. The only cure is to restart the computer. Unfortunately this is hard to reproduce.

cailafinn pushed a commit that referenced this issue Jun 1, 2020
@martyngigg @cailafinn
Convert ConfigService paths to absolute on demand
eb1faa1 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs #12425
cailafinn pushed a commit that referenced this issue Jun 2, 2020
@martyngigg @cailafinn
Convert ConfigService paths to absolute on demand
95ca768 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs #12425
cailafinn pushed a commit that referenced this issue Jun 4, 2020
@martyngigg @cailafinn
Convert ConfigService paths to absolute on demand
d8205f1 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs #12425
cailafinn pushed a commit that referenced this issue Jun 10, 2020
@martyngigg @cailafinn
Convert ConfigService paths to absolute on demand
77385d0 
If the key contains the string "director" it is assumed to point
to a directory and if it its value is relatiove then the path
is converted to an absolute path. This reduces the workload on
startup while keeping the caching of important keys
such as the datasearch paths and instrument directories for performance.
Refs #12425
@martyngigg
Copy link
Member

Now that #28760 is merged I'm going to close this and we can open more specific issues in the future such as #28822 when/if we find other problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cailafinn cailafinn

Labels
Framework Issues and pull requests related to components in the Framework ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Projects
None yet
Milestone
Release 5.1 S3
Development

No branches or pull requests
5 participants
@martyngigg @abuts @gemmaguest @gvardany @cailafinn