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.

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

show warning messages if the calculation takes too long time #816

Closed
superstar54 opened this issue Sep 26, 2024 · 1 comment · Fixed by #840
Closed

show warning messages if the calculation takes too long time #816

superstar54 opened this issue Sep 26, 2024 · 1 comment · Fixed by #840
Assignees
Labels
enhancement New feature or request

Comments

@superstar54
Copy link
Member

superstar54 commented Sep 26, 2024

Is it possible to show warning messages e.g. if people select a combination of parameters that would make calculations too slow?
Can defaults be changed if we are in the demo server? If so, Carlo and Giovanni will recommend different defaults (and possibly different messages to show in some cases) E.g., clean workdir, and default protocol = Fast, and ‘structure as is’. If we can, Carlo and Giovanni can suggest when to write, and what. [@jason @edan @xing]. Add a warning (in step 3) if the calculation takes too long time, e.g. running 5 atoms in a localhost with 2 cpus.

Or more memory

@superstar54 superstar54 added the enhancement New feature or request label Sep 26, 2024
@mikibonacci
Copy link
Member

see also #737

mikibonacci added a commit that referenced this issue Oct 8, 2024
This fixes #816

* Displaying the warning message if resources are not enough.

The check and the message were already there, but not triggered/shown.

* Modification for the warning message.

* Adding also volume check, for low dimensional system can be useful.

* Refining warning messages

- not on localhost but big system and CPUs<4
- same as above, but on localhost
- on localhost and more than 1 CPUs (we may not have enough slots)

* adding self.input_structure check

* Warning messages modified as suggested

* Adding tests for warning messages

---------

Co-authored-by: AndresOrtegaGuerrero <34098967+AndresOrtegaGuerrero@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants