OpenFreeEnergy / IndustryBenchmarks2024

Repository for the 2024 OpenFE industry benchmark efforts
https://industrybenchmarks2024.readthedocs.io/en/latest/
MIT License
1 stars 5 forks source link

fix up run script to match quickrun better #76

Closed IAlibay closed 3 weeks ago

IAlibay commented 3 weeks ago

Fixes #72

Input submission checklist

For each submitted directory:

Summary of changes

Licensing agreement

pep8speaks commented 3 weeks ago

Hello @IAlibay! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 30:80: E501 line too long (156 > 79 characters) Line 39:80: E501 line too long (88 > 79 characters) Line 190:80: E501 line too long (84 > 79 characters) Line 192:1: W293 blank line contains whitespace Line 216:80: E501 line too long (84 > 79 characters)

Comment last updated at 2024-06-13 19:52:42 UTC
mikemhenry commented 3 weeks ago

Oh and one more thing, we need to have some " around a few of those variables if there is a chance that spaces could end up in the file names, but if that can't happen we should be good!