Skip to content

Origin/protein design#21

Open
ChristineSchulz wants to merge 32 commits intomainfrom
origin/protein_design
Open

Origin/protein design#21
ChristineSchulz wants to merge 32 commits intomainfrom
origin/protein_design

Conversation

@ChristineSchulz
Copy link
Collaborator

  • added notebooks / tutorial / pytest for bindcraft, boltz, boltzgen, RFDiffusion on bwVisu
  • adapted website for longer content in the tutorials
  • adapted other website pages accordingly
  • added cute favicon

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

Copy link
Member

@iulusoy iulusoy left a comment

Choose a reason for hiding this comment

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

I like the re-structuring of the resources and the new tutorials.
So far, I have only reviewed the bindcraft one, but am already submitting the comments thus far.
I have no idea why and when I should use bindcraft, and what all the output files are. Maybe it could be useful to add one or two sentences about this. Other than that, the tutorial worked very smoothly.

To use Bindcraft, we need to request a GPU core of type A40.
Choose the Kernel Path to Bindcraft: `/mnt/sds-hd/sd25g005/bindcraft/share/jupyter/` [Contact us](/contact.md) for access to this shared directory.

![Screenshot](../images/tutorial/bwVisu_GPU_Kernel.png)
Copy link
Member

Choose a reason for hiding this comment

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

I understand that you do not want to place images for each tutorial for this in the repo, but I think some users may miss the point about the kernel path. So I would at least also mark the kernel path with a red square, if not put the correct path in the box and take another screenshot.

Copy link
Member

Choose a reason for hiding this comment

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

I think you already changed this in the meantime, so please disregard the comment.


### Step 2: Connect to bwVisu and Start Jupyter

Go to [https://bwvisu.bwservices.uni-heidelberg.de/](https://bwvisu.bwservices.uni-heidelberg.de/ ) and log in with your credentials and one-time password. Please note that you need to be connected to Heidelberg University's VPN if you are connecting from outside the campus.
Copy link
Member

Choose a reason for hiding this comment

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

The link opens in the same tab when using markdown syntax. You could also consider using html for the link and then force it to open in a new tab, for better user experience. I am not sure if this would apply to any other link.

![Screenshot](../images/tutorial/bwVisu_Bindcraft_modules_loaded.png)
{: style="width:378px"}

In the notebook you can check the modules by checking the output of `! module list` which should look like that:
Copy link
Member

Choose a reason for hiding this comment

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

Step 4 Open the notebook

Then check the modules and imports

Start the calculation is then Step 5

"metadata": {},
"outputs": [],
"source": [
"BINDCRAFT_WORKING_DIR = Path.home() / \"protein_design_w_Bindcraft\""
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"BINDCRAFT_WORKING_DIR = Path.home() / \"protein_design_w_Bindcraft\""
"BINDCRAFT_WORKING_DIR = Path.cwd()

Copy link
Member

Choose a reason for hiding this comment

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

Because otherwise everyone has to place it in exactly the same folder in home, which I did not do and of course creates issues. Even if you tell people to do it this way, they will not for some reason and then get stuck.

@iulusoy
Copy link
Member

iulusoy commented Feb 13, 2026

Why is the pre-commit hook nbstripout commented out?

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