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

Limit compute available for each session #26

Closed
ChrisWellsWood opened this issue Feb 15, 2021 · 2 comments · Fixed by #74
Closed

Limit compute available for each session #26

ChrisWellsWood opened this issue Feb 15, 2021 · 2 comments · Fixed by #74
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@ChrisWellsWood
Copy link
Contributor

Currently, a user could dump 1,000 structure files with 1,000,000 residues each into de-stress and the server would fall over. Although I trust our future users, it's probably not ideal behaviour. There are a few ways that we can tackle this:

  1. Limit the number of files that can be running on the server for an individual session
  2. Limit the number of residues allowed in a structure file
  3. Add a time limit to individual jobs running on the server

These are not mutually exclusive and I think we should probably use all of these.

@ChrisWellsWood ChrisWellsWood added the enhancement New feature or request label Feb 15, 2021
@ChrisWellsWood ChrisWellsWood added this to the v0.1.0 milestone Feb 15, 2021
@ChrisWellsWood ChrisWellsWood self-assigned this Feb 15, 2021
@ChrisWellsWood
Copy link
Contributor Author

@MichaelJamesStam ran a test where he tried to load 400 models into the application and the UI became unresponsive. This could be due to the debugger. Worth investigating to determine if we can improve this.

@ChrisWellsWood
Copy link
Contributor Author

One solution would be to throttle job submissions client side if too many jobs are submitted, rather than flooding the queue.

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.

1 participant