Skip to content

Add SIFT#205

Open
arteemg wants to merge 5 commits into
ProteinGym:mainfrom
arteemg:main
Open

Add SIFT#205
arteemg wants to merge 5 commits into
ProteinGym:mainfrom
arteemg:main

Conversation

@arteemg

@arteemg arteemg commented Jul 1, 2026

Copy link
Copy Markdown

No description provided.

@karel-w

karel-w commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Hey Arteemg, thanks for your contibution to ProteinGym.

To be able to merge the models in can you check the following:

  • when opening a PR can you follow the auto populated template? If you didn't get the auto populated template on opening a PR please let me know.
  • If the PR is still in progress, can you convert it to a draft? There is a button for this under requesting reviewers.
  • You are currently missing the train entrypoint that we access with our DVC script, an example of the latest entrypoint can be viewed here

@arteemg

arteemg commented Jul 6, 2026

Copy link
Copy Markdown
Author

Hi @karel-w, thank you for your reply! What is the auto-populated template? I haven't been able to find it.

@karel-w

karel-w commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Hi @karel-w, thank you for your reply! What is the auto-populated template? I haven't been able to find it.

When you open a PR it should give some pre-filled text with a section to describes changes and a checklist to go over your changes? Maybe this doesn't happen if the PR comes from a fork?

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