Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GromacsMDSetupUnit #7

Merged
merged 94 commits into from
Sep 20, 2024
Merged

GromacsMDSetupUnit #7

merged 94 commits into from
Sep 20, 2024

Conversation

hannahbaumann
Copy link
Contributor

Description

Provide a brief description of the PR's purpose here.

Todos

Notable points that this PR has either accomplished or will accomplish.

  • TODO 1

Questions

  • Question1

Status

  • Ready to go

@pep8speaks
Copy link

pep8speaks commented Jul 16, 2024

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

Line 4:80: E501 line too long (80 > 79 characters)
Line 29:80: E501 line too long (80 > 79 characters)
Line 63:80: E501 line too long (86 > 79 characters)
Line 184:80: E501 line too long (80 > 79 characters)
Line 281:80: E501 line too long (82 > 79 characters)
Line 418:80: E501 line too long (84 > 79 characters)
Line 511:80: E501 line too long (81 > 79 characters)
Line 515:80: E501 line too long (87 > 79 characters)
Line 549:80: E501 line too long (85 > 79 characters)
Line 575:80: E501 line too long (81 > 79 characters)
Line 582:80: E501 line too long (85 > 79 characters)
Line 626:80: E501 line too long (82 > 79 characters)
Line 671:80: E501 line too long (84 > 79 characters)

Line 78:80: E501 line too long (84 > 79 characters)
Line 208:80: E501 line too long (83 > 79 characters)
Line 210:80: E501 line too long (87 > 79 characters)

Line 94:80: E501 line too long (80 > 79 characters)
Line 115:80: E501 line too long (82 > 79 characters)
Line 117:80: E501 line too long (86 > 79 characters)
Line 146:80: E501 line too long (82 > 79 characters)
Line 148:80: E501 line too long (86 > 79 characters)
Line 162:80: E501 line too long (81 > 79 characters)

Line 47:80: E501 line too long (91 > 79 characters)
Line 114:80: E501 line too long (80 > 79 characters)
Line 138:80: E501 line too long (80 > 79 characters)
Line 183:80: E501 line too long (83 > 79 characters)

Line 15:80: E501 line too long (88 > 79 characters)

Comment last updated at 2024-09-20 10:21:06 UTC

@hannahbaumann hannahbaumann changed the title Start of GromacsMDSetupUnit [WIP] Start of GromacsMDSetupUnit Jul 16, 2024
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
openfe_gromacs/protocols/gromacs_md/md_methods.py Outdated Show resolved Hide resolved
@hannahbaumann hannahbaumann changed the title [WIP] Start of GromacsMDSetupUnit GromacsMDSetupUnit Sep 20, 2024
@hannahbaumann hannahbaumann merged commit 3124bc1 into main Sep 20, 2024
7 of 9 checks passed
@hannahbaumann hannahbaumann deleted the start_GromacsMDProtocol branch September 20, 2024 10:26
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.

4 participants