remove build strings from env to make it more portable and add UI - #51
Open
duerrsimon wants to merge 3 commits into
Open
remove build strings from env to make it more portable and add UI#51duerrsimon wants to merge 3 commits into
duerrsimon wants to merge 3 commits into
Conversation
Author
|
Added the UI to run the predictions locally to this pull request. |
|
Hi, by chance do you have a standalone UI to review the output? (without needing al dependencies related to the inference). |
Author
|
if you install gradio and gradio-molecule3d on your remote machine you will be able to start the computations and view the output locally thanks to the link sharing feature of gradio. |
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
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.
environment.yml changes:
The environment.yml contains build strings for all conda packages.
When I was trying to install conda claimed it will take 2h+ to fix possible conflicts because the environment was inconsistent.
We're running pretty standard Debian Linux 11 workstations in the lab so it should not take this long to install the environment. Especially in HPC environments the currently required builds of the requested packages might not fit with the specs of the machine.
Removing the build strings allows the package manager to pick builds of the requested packages that match the specific hardware/os configuration better.
After I removed the build strings the environment installed without any error in a few minutes. Inference works perfectly too.
BTW: The environment would become even more portable by using
conda env export --from-history --no-buildsSome background here: https://stackoverflow.com/a/56780297
New UI for generating input and running predictions
This pull request also adds a UI that can be launched using

python ui.pyfrom the main directory.Run locally the UI will either generate the input files like https://hf.co/spaces/simonduerr/RFAA (dry run) or also run the prediction and output the predicted structure plus plots of the pLDDT, PAE and PDE.