Skip to content

Implemented seed argument for reproducibility (redo) - #71

Open
nzrandol wants to merge 2 commits into
RosettaCommons:mainfrom
nzrandol:seed
Open

nzrandol wants to merge 2 commits into
RosettaCommons:mainfrom
nzrandol:seed

Conversation

@nzrandol

Copy link
Copy Markdown
Member

Hello! Thanks for this great tool! I figured I'd try my hand at contributing a little bit. I noticed this issue #5, and I decided to implement this.

The Details:

I added the new seed argument (and a comment explaining it) to base.yaml
seed will affect run_inference.py by being an argument for make_deterministic.

Please let me know if there is anything else I can do!

@mmaavc

mmaavc commented Jan 20, 2025

Copy link
Copy Markdown

Is there any update on merging this? It would be a very useful feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants