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 application compute #74

Merged
merged 3 commits into from
Mar 16, 2021
Merged

Limit application compute #74

merged 3 commits into from
Mar 16, 2021

Conversation

ChrisWellsWood
Copy link
Contributor

In order to limit the amount of compute that a single user can use, I have added the following restrictions to the application:

  1. A hard limit of 500 residues in a single structure file. As most of the algorithms do not scale linearly, the amount of compute required to calculate metrics increases steeply with the number of residues. 500 is bigger than most designed proteins.
  2. A maximum of 30 structures can be loaded at a single time. This is to prevent user error as much as to limit the strain on the server. Previously, if the user tried to upload 1000 files, the application would have went along with that. Now the user must load files in batches of 30.
  3. A timeout of 30 seconds has been added to the job queue to ensure that the queue isn't blocked by badly behaved jobs. I was tempted to limit this even further, so we need to keep an eye on the average job time.

Closes #26

@ChrisWellsWood ChrisWellsWood added the enhancement New feature or request label Mar 16, 2021
@ChrisWellsWood ChrisWellsWood added this to the v0.1.0 milestone Mar 16, 2021
Copy link
Collaborator

@MichaelJamesStam MichaelJamesStam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works really well! I did a test by trying to upload 400 antibody designs and the web server coped with it fine! Only 30 of them were processed and a lot of them were not ran because they exceeded the residue limit of 500.

@MichaelJamesStam MichaelJamesStam merged commit b86ead6 into master Mar 16, 2021
@MichaelJamesStam MichaelJamesStam deleted the limit-compute branch March 16, 2021 10:53
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 this pull request may close these issues.

Limit compute available for each session
2 participants