-
Notifications
You must be signed in to change notification settings - Fork 46
Mpvenkatesh/az #138
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
Open
mpvenkatesh
wants to merge
33
commits into
dev
Choose a base branch
from
mpvenkatesh/az
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Mpvenkatesh/az #138
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
With this change intermediate results are stored in sqlite database. All results from generative models are stored in SQLite database. With this change, we reduce the total number of requests to MegaMolbart gRPC service by 3/4th. Additionally, training dataset is loaded into SQLite database and used while computing Novelity metric. With this change, a request to check Novelity is 2ms. Other changes include: - Upgrade to rapids 2021.06 - Clean dockerfile to remove all workarrounds - Remove the need for conda cuchem env inside the container. - Ability to select docker image to build using launch script - Add smile2embedding and embedding2smile to cddd - Use hydra for benchmark configuration
RAPIDS 21.08 has fix for a bug filed for PCA on Multiple GPUs. Other changes include: - 'GPU KMeans-UMAP - Single and Multiple GPUs' is now the default clustering workflow. This one has performance benefit compare to 'GPU KMeans-UMAP' while reclustering. - 'MegatronMolBART' is now the default generative workflow - Remove Apex installation while docker image creation.
Change directory names
…flagged by github
Now docker-compose will be downloaded the first time the application is launched. Another changes is upgrade ansible script to install on ubuntu 20.04.
Also remote clear cache option while building docker image.
rilango
approved these changes
Mar 3, 2022
Author
|
@rilango , I believe my branch is ready for the merge with dev. Please let me know if you have any suggestions or comments. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The features previously implemented in the az repo have been incorporated into this cheminformatics repo. These include: