dauparas / ProteinMPNN

Code for the ProteinMPNN paper
MIT License
934 stars 284 forks source link

Suppress warning when using release version. #33

Closed roccomoretti closed 1 year ago

roccomoretti commented 1 year ago

When running protein_mpnn_run.py on a non-Git copy (e.g. the released versions), a rather concerning error message gets printed:

fatal: Not a git repository: '/path/to/ProteinMPNN/.git'

Despite the message, this is not a fatal error for ProteinMPNN.

Since the script handles the issue, we can simply suppress the printing of the error by discarding stderr.

There probably should be a better mechanism to insert the "official" release version here, but failing that suppressing the error message is a decent user-friendliness stopgap.

dauparas commented 1 year ago

Thanks for pointing this out!