Skip to content

Rosettafold addition#53

Merged
hlasimpk merged 14 commits into
mainfrom
rosettafold_addition
Apr 28, 2026
Merged

Rosettafold addition#53
hlasimpk merged 14 commits into
mainfrom
rosettafold_addition

Conversation

@hlasimpk
Copy link
Copy Markdown

Added code to install, run, and produce output files from RosettaFold3
I've also added test data, tests and updated documentation.

This should be fairly straightforward as it was easy enough to implement using existing code/strategies.

@hlasimpk hlasimpk requested a review from hllelli2 April 15, 2026 15:55
with open(file_path, "w") as f:
f.write(msa)

def json_to_json(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe make this name a bit more descriptive!

Comment thread abcfold/rosettafold3/check_install.py Outdated
def ensure_rosettafold_checkpoint(target_path: Path) -> Path:
"""
Ensure RoseTTAFold3 checkpoint exists at target_path.
Non-interactive, automation-safe.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reads like an AI docstring, not needed.

Copy link
Copy Markdown
Collaborator

@hllelli2 hllelli2 left a comment

Choose a reason for hiding this comment

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

Looks good to me. See the minor comments I've made but good to go

@hlasimpk hlasimpk merged commit dc08068 into main Apr 28, 2026
5 of 6 checks passed
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