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

Performace Improvement on the run CLI(ersilia-os#1299) #1351

Closed
wants to merge 33 commits into from

Conversation

Abellegese
Copy link
Contributor

Thank you for taking your time to contribute to Ersilia, just a few checks before we proceed

  • Have you followed the guidelines in our Contribution Guide
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Description

Overall performance of the run CLI has been increased substantially by making the changes below(comparison results will be given at #1299)

  1. Asynchronous and concurrent based key converter at the compound identifier
  2. Caching(with dynamic custom cache size) key conversion results for smiles both from the APIs and from rdkit (main performance boost)
  3. Adding a valid smiles checker
  4. api schema and information file reader function improvement
  5. reduction in the unnecessary heavy app layers
  6. more

Related to #1299

Abellegese and others added 30 commits October 9, 2024 22:15
@Abellegese Abellegese closed this Oct 26, 2024
@Abellegese Abellegese reopened this Oct 26, 2024
@Abellegese Abellegese closed this Oct 26, 2024
@Abellegese Abellegese deleted the improv-2 branch October 26, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants